On 12/23/2010 01:31 AM, Laine Stump wrote:
+ int nips; s/int/size_t/
Okay, I've changed it. I'll point out that the great majority of the "n<thing>s" variables in *_conf.h are defined as int (and some more as unsigned int). Should these all be standardized at some point?
Probably, but not high priority.
I was going to attach a delta diff, but realized after the fact that I didn't know how to get a diff between an old and new version of a commit once I'd rebased. Instead, I'm pasting the new regexp below for you to review; that's the only significant change. The others have all been squashed in as well.
Fair enough. Agree to ACK for now, and we can further fix the regex later (to fix the noted problem with accepting :1.2.3.4 - too loose rng is not a show-stopper for actually using valid IPv6 in XML) [hmm - can you tell that this is a big enough series that I'd like to get it in before 0.8.7 to widen the test coverage?]. -- Eric Blake eblake@redhat.com +1-801-349-2682 Libvirt virtualization library http://libvirt.org