Re: [PATCH] Add color.diff.{header,message} formats

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

 



Pierre Habouzit <madcoder@xxxxxxxxxx> writes:

>   Hmmm it seems this patch has been somehow lost somewhere, is there
> anything wrong with it ?

It may not break anything, and I did not hear anybody on the list who said
this breaks soemthing.  But I did not hear anybody on the list who wanted
to have this except you, either.  At this stage of development, the
question we should be asking is not if there is anything wrong with it,
but if there is anything _good_ with it.

It looks rather widespread compared to a perceived benefit.  Perhaps after
a refactoring to make the parameter to pretty-print-commit a bit more
easily extensible, the patch might start looking more promising.  I dunno.




--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[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