cvs commit: src/usr.sbin/pkg_install/lib deps.c

Kris Kennaway kris at obsecurity.org
Wed Jun 20 02:01:39 UTC 2007


On Tue, Jun 19, 2007 at 04:26:54PM -0700, Maxim Sobolev wrote:
> Kris Kennaway wrote:
> >On Tue, Jun 19, 2007 at 04:22:35PM -0700, Maxim Sobolev wrote:
> >>Kris Kennaway wrote:
> >>>On Tue, Jun 19, 2007 at 04:16:48PM -0700, Maxim Sobolev wrote:
> >>>>I have made the following patch to bring style(9) back and make code 
> >>>>more readable. Do you guys want to run it on bento or could I just 
> >>>>check it in?
> >>>>
> >>>>http://sobomax.homeunix.org/~sobomax/deps.c.diff
> >>>Is md5 checksum the same before/after?
> >>Well, it has some trivial non-whitespace changes.
> >
> >whitespace in source does not show up in the binary.
> 
> Please re-read my message. It has NON-whitespace changes as well.

So the answer to my question is ... ?  I'm still not sure you
understood it.  If you are making no functional changes then the md5
of the resulting binary will be unchanged, and this is in fact a good
way to check whether you are introducing functional changes or just
style/whitespace changes.

Kris
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 187 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/cvs-src/attachments/20070620/561c6b8e/attachment.pgp


More information about the cvs-src mailing list