Re: [PATCH v3] deprecate fclose() and introduce VIR_{FORCE_}FCLOSE()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 11/16/2010 10:37 AM, Eric Blake wrote:
On 11/16/2010 04:50 AM, Stefan Berger wrote:
V3:
   - fixes from V2 review + one lost hunk

   ->   diff(tree+V2, tree+v3) at bottom of email
That helps.  And you solved all my findings.

ACK.

+++ src/xen/xen_driver.c    2010-11-16 06:31:49.722481846 -0500
@@ -218,9 +218,9 @@
  #endif
  #ifdef __sun
      int fd;

-    if (fd = open("/dev/xen/domcaps", O_RDONLY)) {
+    if ((fd = open("/dev/xen/domcaps", O_RDONLY))>= 0) {
Did you want a space before>=?

It was with the space in my patch locally. It must have been mangled by Thunderbird again. Evolution keeps on crashing on me lately...

Pushed.

   Stefan

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]