https://bz.apache.org/bugzilla/show_bug.cgi?id=57045

--- Comment #15 from Eric Covener <[email protected]> ---
(In reply to Kurtis Rader from comment #14)
> "Can you please post changes as diff -u output?"
> 
> I did. See the first attachment:
> https://bz.apache.org/bugzilla/attachment.cgi?id=33179
> 
> I posted the entire module to make it easy for someone to see the entirety
> of the change since the patch is twice the length of the original module.

Sorry, assumed it superceded.  

It looks like a large part of the diff (other than that there's info for 2
copies of the mod?) is refactoring and changing C comments to C++/c99 comments.
 Noise aside, the latter should be avoided for modules. The
refactoring/all-in-one patch makes it a lot of labor to review.

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to