[darcs-devel] [patch1775] Assorted collection of minor code cleanups (18 patches)

[Ben Franksen]

Ben Franksen <ben.franksen at online.de> added the comment:

Am 02.06.19 um 20:37 schrieb Ganesh Sittampalam:
> 
> Ganesh Sittampalam <ganesh at earth.li> added the comment:
> 
>>  * group --reorder under the merge options and improve its help 
> text
> 
>> +  [ RawNoArg [] ["reorder-patches"] F.Reorder Reorder
>> +    "put local-only patches on top of remote ones"
>> +  , RawNoArg [] ["no-reorder-patches"] F.NoReorder NoReorder
>> +    "put remote-only patches on top of local ones" ]
> 
> I think the comment about NoReorder is a bit misleading. Reorder
> explicitly moves all local-only patches to the top of the
> repository. But NoReorder will just put the *new* remote patches
> at the top of the repository, leaving everything else in the 
> original order (which could have been anything).

Good catch. Perhaps this is better:

"place new patches at the end of your repository"

?

__________________________________
Darcs bug tracker <bugs at darcs.net>
<http://bugs.darcs.net/patch1775>
__________________________________