Re: patch for macOS screenshot dir (Was: [hatari-devel] Screenshots are saved in Hatari installation directory)

[ Thread Index | Date Index | More lists.tuxfamily.org/hatari-devel Archives ]


Le 25/02/2018 à 20:30, Thomas Huth a écrit :
Am Sun, 25 Feb 2018 13:27:58 +0100
schrieb Nicolas Pomarède <npomarede@xxxxxxxxxxxx>:

Le 19/02/2018 à 12:04, Nicolas Pomarède a écrit :
Le 17/02/2018 à 01:05, Troed Sångberg a écrit :
Hi all,

Thanks for the quick testing! Patch attached.
[...]

patch applied to main sources tree (I just added a check before the
"free" in Paths_Init() )

Just FYI: free() has always a built-in NULL-pointer check, so you don't
need to check that a pointer is != NULL before calling free().

you're right, but I kept this habit from much older version I think, where NULL pointer had bad side effect :)




Mail converted by MHonArc 2.6.19+ http://listengine.tuxfamily.org/