|
From: <sv...@va...> - 2008-06-29 08:42:43
|
Author: bart
Date: 2008-06-29 09:42:46 +0100 (Sun, 29 Jun 2008)
New Revision: 8308
Log:
Added support for %', next to %,.
Modified:
branches/FORMATCHECK/coregrind/m_debuglog.c
Modified: branches/FORMATCHECK/coregrind/m_debuglog.c
===================================================================
--- branches/FORMATCHECK/coregrind/m_debuglog.c 2008-06-29 08:10:22 UTC (rev 8307)
+++ branches/FORMATCHECK/coregrind/m_debuglog.c 2008-06-29 08:42:46 UTC (rev 8308)
@@ -652,7 +652,7 @@
i++;
}
/* If ',' follows '%', commas will be inserted. */
- if (format[i] == ',') {
+ if (format[i] == ',' || format[i] == '\'') {
flags |= VG_MSG_COMMA;
i++;
}
|
|
From: Julian S. <js...@ac...> - 2008-06-29 08:51:29
|
On Sunday 29 June 2008 10:42, sv...@va... wrote: > Author: bart > Date: 2008-06-29 09:42:46 +0100 (Sun, 29 Jun 2008) > New Revision: 8308 > > Log: > Added support for %', next to %,. If %, already gives that functionality, why is %' also necessary? J |
|
From: Bart V. A. <bar...@gm...> - 2008-06-29 08:58:26
|
On Sun, Jun 29, 2008 at 10:44 AM, Julian Seward <js...@ac...> wrote: >> Log: >> Added support for %', next to %,. > > If %, already gives that functionality, why is %' also necessary? Because gcc's format string checking understands %' but not %,. Bart. |