]> granicus.if.org Git - graphviz/commit
remove fallback "%?%K" strftime format in expr’s printf
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Sun, 8 Aug 2021 22:20:11 +0000 (15:20 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Thu, 12 Aug 2021 14:48:36 +0000 (07:48 -0700)
commit12fd48bd846a44e05a03a007beeb848d6de7604c
tree7c01eb0fc8232603375333c48bab6b4f90284247
parent07c714e9c150fbba026b652ccd453b210a5f265a
remove fallback "%?%K" strftime format in expr’s printf

lib/expr’s printf implementation supports a "%t" format code to print a time. If
no format is provided for this, it would fallback to a default of "%?%K".

While working on #1998, one of my intermediate refactorings moved this string
literal into the actual call to strftime. At this point the compiler politely
interjected that neither "%?" nor "%K" are format strings understood by
strftime. So it seems this results in an invalid call to strftime.

To deal with this, we now throw an error when this function is called with no
format string instead of making an invalid strftime call.
CHANGELOG.md
lib/expr/exeval.c