Re: [GSoC][PATCHl 1/6] rebase -i: add --ignore-whitespace flag

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Phillip

On Thu, 8 Aug 2019 17:44:38 +0100 Phillip Wood <phillip.wood123@xxxxxxxxx> wrote:
> 
> [...]
> >   --ignore-whitespace::
> > +	This flag is either passed to the 'git apply' program
> > +	(see linkgit:git-apply[1]), or to 'git merge' program
> > +	(see linkgit:git-merge[1]) as `-Xignore-space-change`,
> > +	depending on which backend is selected by other options.
> 
> I think it would be better to document the effect of this option rather
> than the implementation detail. It is confusing at the moment as it
> talks about 'git merge' but we don't allow this option with merges.

Oh, it is just to indicate the user where to look for the definitions
as currently the behaviour of both the backends is not exactly the
same neither do they work optimally. Hope in future when they are in
harmony, then we can happily change it to match the then behaviour.

> [...]

Thanks
Rohit




[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux