On Fri, Feb 10, 2017 at 03:50:36PM +0100, Pavel Grunt wrote: > On Thu, 2017-02-09 at 18:46 +0100, Christophe de Dinechin wrote: > > I tend to agree. I don’t see how any legitimate GError * (from > > g_error_new or the like) would have a NULL message. So this is > > really defensive coding. > > > nothing prevents calling g_error_new with a NULL message :) I know it > is not our case > > > Alternate solution would be to assert, but I don’t like killing the > > viewer if this ever happens. > > > > > No need, I think > const char *str = NULL; > g_print("%s", str); > does not crash and prints (null) Bit of old trivia, on Solaris this (used to?) crashes Christophe
Attachment:
signature.asc
Description: PGP signature
_______________________________________________ virt-tools-list mailing list virt-tools-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/virt-tools-list