Re: CVS commit: src/usr.bin/make

2015-10-06 Thread Christos Zoulas
In article <3c6cf536-97db-4dae-966e-a3c02c505...@msys.ch>, Marc Balmer wrote: >>v->name, delim); >> -if (*freePtr) { >> -free(*freePtr); >> -*freePtr = NULL; >> -} >> +free(*freePtr); >> +*freePtr = NULL; > >is this sentinel, setting *freePtr to NULL, really n

Re: CVS commit: src/usr.bin/make

2015-10-06 Thread Marc Balmer
> Am 06.10.2015 um 19:36 schrieb Christos Zoulas : > > Module Name: src > Committed By: christos > Date: Tue Oct 6 17:36:25 UTC 2015 > > Modified Files: > src/usr.bin/make: var.c > > Log Message: > don't check for NULL before free() (Tilman Sauerbeck) > > > To generate a diff

Re: CVS commit: src/sys/arch/arm/allwinner

2015-10-06 Thread Jared McNeill
On Tue, 6 Oct 2015, Manuel Bouyer wrote: On Tue, Oct 06, 2015 at 10:06:31AM +1100, matthew green wrote: And is this really a problem ? turning off video is configurable in the X server, so if you want audio without video you can turn off the screensaver ... it's a problem if you want the scre

Re: CVS commit: src/sys/arch/arm/allwinner

2015-10-06 Thread Manuel Bouyer
On Tue, Oct 06, 2015 at 10:06:31AM +1100, matthew green wrote: > > And is this really a problem ? turning off video is configurable in the > > X server, so if you want audio without video you can turn off the > > screensaver ... > > it's a problem if you want the screensaver to work when you're no