|
From: Paul M. <pm...@sk...> - 2012-10-26 16:26:03
|
Sorry, looks like I managed to reply directly to Kevin rather than the list.
Here's the patch submission.
-Paul
Kevin / Sebastian -- This code change has lots of parts, so I want to make the
changes incrementally so that you can review the code changes and commit them
(if they're acceptable) as I go.
Attached please find the first patch. It's as I described below for #1 and #2,
except I went with "mInitialized" and "isInitialized()" for readability. Note
that this is the opposite sense of the deprecated "mEmpty", so much of the patch
is devoted to swapping true and false, and adding/removing "!" operators, for
the existing Box code.
Let me know if this flies (I expect it well, as it's the most trivial of the
changes), then I'll proceed with the other mods. Thanks!
-Paul
|