gnucash master: Fix wrong PWARN calling signature.
John Ralls
jralls at ceridwen.us
Tue Jul 12 14:31:49 EDT 2016
> On Jul 12, 2016, at 11:14 AM, Aaron Laws <dartme18 at gmail.com> wrote:
>
> On Tue, Jul 12, 2016 at 2:13 PM, Aaron Laws <dartme18 at gmail.com> wrote:
>
>>
>> On Tue, Jul 12, 2016 at 12:57 PM, John Ralls <jralls at code.gnucash.org>
>> wrote:
>>
>>> Updated via https://github.com/Gnucash/gnucash/commit/01c21da3 (commit)
>>> from https://github.com/Gnucash/gnucash/commit/c11185e9 (commit)
>>>
>>>
>>>
>>> commit 01c21da3234ba637f57cc4b18eb069ef2bcc7aa9
>>> Author: John Ralls <jralls at ceridwen.us>
>>> Date: Tue Jul 12 16:57:01 2016 +0000
>>>
>>> Fix wrong PWARN calling signature.
>>>
>>>
>>>
>>> Summary of changes:
>>> src/libqof/qof/gnc-timezone.cpp | 2 +-
>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> _______________________________________________
>>> gnucash-patches mailing list
>>> gnucash-patches at gnucash.org
>>> https://lists.gnucash.org/mailman/listinfo/gnucash-patches
>>>
>>
>> - PWARN(str.str().c_str());
>> + PWARN("%s", str.str().c_str());
>> What's the difference here?
>>
>>
>
> I mean, I see that if the string is, for instance, "%s", nothing is output.
> But did this come up, or did you catch it visually while looking through
> the code.
master's nightly failed. I thought I'd tested it before committing, but when I tried again this morning it failed.
The issue is that gcc will happily a single-argument invocation of PWARN(format, ...) but g++ --at least the version in MinGW--won't.
Regards,
John Ralls
More information about the gnucash-devel
mailing list