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

[Nicolas Pomarède <npomarede@xxxxxxxxxxxx>]

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 :)