On Fri, Sep 21, 2012 at 11:37:08AM +0200, Michal Privoznik wrote: > On 21.09.2012 11:27, Michal Privoznik wrote: > > On 20.09.2012 17:29, Richard W.M. Jones wrote: > >> From: "Richard W.M. Jones" <rjones@xxxxxxxxxx> > >> > >> This allows the user to control labelling of each character device > >> separately (the default is to inherit from the VM). > >> --- > >> docs/formatdomain.html.in | 8 ++++ > >> src/conf/domain_conf.c | 77 +++++++++++++++++++++++++++++++++-- > >> src/conf/domain_conf.h | 6 +++ > >> src/security/security_selinux.c | 86 +++++++++++++++++++++++++++------------ > >> 4 files changed, 147 insertions(+), 30 deletions(-) > >> > > Ah, I gave premature ACK; you should have updated RNG as well. But I > believe you can handle it without any special review. So my ACK holds as > long as you update the schema and coding style raised by the first review. I'll post an updated version anyway. Are there style guidelines on long lines? I'm seeing a lot of long lines in the existing code .. Rich. -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones New in Fedora 11: Fedora Windows cross-compiler. Compile Windows programs, test, and build Windows installers. Over 70 libraries supprt'd http://fedoraproject.org/wiki/MinGW http://www.annexia.org/fedora_mingw -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list