extracting an orthogonal cleanup from patch1911

1 patch for repository darcs-unstable@darcs.net:screened:

patch 930c67427998afb9099a5dd479482e087d5b4ef3
Author: Ganesh Sittampalam <ganesh@earth.li>
Date:   Sat Sep 14 16:38:26 BST 2019
  * rename D.P.Rebase.Container to D.P.Rebase.Suspended
  
  The module primarily contains the Suspended type and code
  to manipulate it.

Attachments

• patch-preview.txt
• rename-d_p_rebase_container-to-d_p_rebase_suspended.dpatch
• unnamed

> 1 patch for repository darcs-unstable@darcs.net:screened:
> 
> patch 930c67427998afb9099a5dd479482e087d5b4ef3
> Author: Ganesh Sittampalam <ganesh@earth.li>
> Date:   Sat Sep 14 16:38:26 BST 2019
>   * rename D.P.Rebase.Container to D.P.Rebase.Suspended
>   
>   The module primarily contains the Suspended type and code
>   to manipulate it.

FWIW, I find the module name Container quite fitting. It describes
clearly what this module gives us: the one big rebase patch that
contains all the patches to be edited and their fixups.

> FWIW, I find the module name Container quite fitting. It describes
> clearly what this module gives us: the one big rebase patch that
> contains all the patches to be edited and their fixups.

Ah, I thought it'd be uncontroversial so already screened it (just forgot
to update the status). I had the same reason for giving it that name to 
begin with, but in retrospect it seemed like it was just gratuitously
making the code harder to explore.

>> FWIW, I find the module name Container quite fitting. It describes
>> clearly what this module gives us: the one big rebase patch that
>> contains all the patches to be edited and their fixups.
> 
> Ah, I thought it'd be uncontroversial so already screened it (just forgot
> to update the status). I had the same reason for giving it that name to 
> begin with, but in retrospect it seemed like it was just gratuitously
> making the code harder to explore.

I understand, but I'd rather have renamed the patch type Suspended to
RebaseContainer (and then perhaps renamed RebaseItem to Suspended). This
implies a lot more churn though, as now some of the function names may
now be named confusingly, so we'd have to rename them, too, etc etc.

My objection stands but it's really not a big deal and I can live
perfectly well with the new module name.