Well, I've been writing software professionally for many years now, and I can assure you that the only true measures of good coding practices are stability, portability, and reuseability. Focusing on such inconsequentials like style conventions and so forth is really just a waste of time, IMO. Developers that yield aesthetically pleasing crap are a dime a dozen; the best ones often produce the most quirkily written code imaginable, but are yet as rare as diamonds and thus worth their weight in gold! Bottom line: substance always trumps style. Full stop.
Just bear in mind that the literal ~0 is an int and so the left-hand-side gets widened to an int before the XOR, then narrowed back to unsigned char for the assignment, which may result in undefined behavior (correct me if I'm wrong). Just to be safe, you should always use explicit casts in such situations. In other words:
Code:*begin++ ^= (unsigned char)~0;