Michael Lucas wrote:
> 
> Hello,
> 
> I'm afraid I might be walking up to a bikeshed with a can of paint
> here, but the flood of email in the last twenty-four hours has
> convinced me to ask.
> 
> In an article O'Reilly published yesterday, I stated (per the
> Handbook) that context diffs were the correct way to submit patches
> with PRs.  I've had several people claim that unified diffs are the
> way to go, and that the handbook is just wrong.
> 
> Is the Handbook correct, or are unified diffs preferred?  I'll be
> happy to fix my article and submit a PR to correct the Handbook if
> this is the case.

Either will do. Unified diffs are less verbose, but sometimes they are
too confusing to be of any use. The point is to use _either_ of them
instead of plain diff (because both of them have "context" -- lines
indicating the point were the changes are to take place).

-- 
Daniel C. Sobral                        (8-DCS)
[EMAIL PROTECTED]
[EMAIL PROTECTED]
[EMAIL PROTECTED]

        It's a rewarding life, but hey, somebody has to have all the fun,
right?

To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-hackers" in the body of the message

Reply via email to