[vbox-dev] [PATCH 2/8] Fix an implicit conversion from pointer to bool

Frank Mehnert frank.mehnert at oracle.com
Thu Mar 26 08:51:50 GMT 2015


Hi,

On Friday 13 March 2015 16:56:56 Jung-uk Kim wrote:
> This fix an implicit conversion from pointer to bool.

why is this patch necessary? An expression like

  if (pointer)
  {
  }

is completely valid and no compiler should warn about it. It tests if
the pointer has a value different from NULL.

Would

-         || (   (a_pVCpu) /*!= NULL*/ \
+         || (   a_pVCpu \

fix the compiler warning as well?

Frank
-- 
Dr.-Ing. Frank Mehnert | Software Development Director, VirtualBox
ORACLE Deutschland B.V. & Co. KG | Werkstr. 24 | 71384 Weinstadt, Germany

Hauptverwaltung: Riesstr. 25, D-80992 München
Registergericht: Amtsgericht München, HRA 95603
Geschäftsführer: Jürgen Kunz

Komplementärin: ORACLE Deutschland Verwaltung B.V.
Hertogswetering 163/167, 3543 AS Utrecht, Niederlande
Handelsregister der Handelskammer Midden-Niederlande, Nr. 30143697
Geschäftsführer: Alexander van der Ven, Astrid Kepper, Val Maher




More information about the vbox-dev mailing list