David Hendricks posted comments on this change.
Patch set 2:Code-Review +1
Either way is fine with me. If you have time and can edit this patch, I'll be happy to re-review and ack. There might be style issues in the follow-up patches, so perhaps doing a single !fixup would be the easiest/quickest thing to do here.
(1 comment)
Patch Set #1, Line 127: + 1) - 1
> I'd eventually like to switch to using dynamically allocated memory
That sounds good to me as well.
To view, visit change 20513. To unsubscribe, visit settings.