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@xxxxxxxxxx +1-801-349-2682 Libvirt virtualization library http://libvirt.org
Attachment:
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list