> On Jan 26, 2019, at 10:57 AM, Alex Aycinena <alex.aycin...@gmail.com> wrote: > >> >> >> ---------- Forwarded message ---------- >> From: John Ralls <jra...@ceridwen.us> >> To: Mike Alexander <m...@umich.edu> >> Cc: "gnucash-devel@gnucash.org" <gnucash-devel@gnucash.org> >> Bcc: >> Date: Fri, 25 Jan 2019 13:01:20 -0800 >> Subject: Re: [GNC-dev] gnucash maint: remove unnecesary 'home' level in >> path for US Income Tax book tax information >> I've just pushed a fix. The non-literal-null-conversion warning is there >> because it's a common mistake to set char *x = '\0' when one means to say >> char x = '\0', so my fix just replaced those with NULL, since Alex wants >> NULL and not "". >> >> Regards, >> John Ralls >> >> >>> On Jan 25, 2019, at 12:12 PM, Mike Alexander <m...@umich.edu> wrote: >>> >>> This commit is causing build failures for me. I get a number of errors >> of the form >>> >>> /tools/gnucash-git/gnucash/libgnucash/app-utils/gnc-ui-util.c:264:73: >>> error: expression which evaluates to zero treated as a null pointer >> constant of type 'const char *' [-Werror,-Wnon-literal-null-conversion] >>> qof_book_set_string_option(book, OLD_OPTION_TAXUS_NAME, >> '\0'); >>> >>> I gather others aren’t seeing this. I’m building with the clang from >> XCode 10.1 which identifies itself as >>> >>> Apple LLVM version 10.0.0 (clang-1000.11.45.5) >>> Target: x86_64-apple-darwin17.7.0 >>> Thread model: posix >>> >>> If no one else is seeing this I’ll try to figure out what is different >> for me. >>> >>> Mike >>> >>>> On Jan 24, 2019, at 9:54 PM, J.Alex Aycinena < >> alex.aycin...@code.gnucash.org> wrote: >>>> >>>> Updated via https://github.com/Gnucash/gnucash/commit/fc153643 >> (commit) >>>> from https://github.com/Gnucash/gnucash/commit/3a486727 (commit) >>>> >>>> >>>> >>>> commit fc1536432619888dc28cd5c0aa58658094a93c45 >>>> Author: Alex Aycinena <alex.aycin...@gmail.com> >>>> Date: Thu Jan 24 18:36:13 2019 -0800 >>>> >>>> remove unnecesary 'home' level in path for US Income Tax book tax >> information >>>> >>>> >>>> >>>> Summary of changes: >>> >>> >> > > > John - Thanks for fixing this. My Fedora 29 system didn't give the error. > > Mike - Is your build working now?
Alex, You're welcome. Gcc doesn't have an equivalent warning so there's no way you could have seen it unless you'd decided to use clang. Regards, John Ralls _______________________________________________ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel
