Re: svn commit: r1042294 - /subversion/trunk/subversion/libsvn_subr/checksum.c
From: Peter Samuelson <peter_at_p12n.org>
Date: Sun, 5 Dec 2010 05:14:20 -0600
Can do. x1 << 4 | x2 didn't seem too ambiguous to me, but I can see
> Can we do one assignment per line instead? Maybe that's easier to parse.
((char *)(*checksum)->digest)[i] = x1 << 4 | x2;
...The repeated expression is complex, so you have to visually verify
...Which doesn't seem much easier to parse either.
This is an archived mail posted to the Subversion Dev mailing list.