diff options
| author | Herbert Xu <herbert@gondor.apana.org.au> | 2014-10-27 15:53:48 +0800 |
|---|---|---|
| committer | Herbert Xu <herbert@gondor.apana.org.au> | 2014-10-27 15:53:48 +0800 |
| commit | 7bb413255368e94395237d789f522891093c5774 (patch) | |
| tree | 2d13bc391beab47e669c799a3af067915f42f619 | |
| parent | [OUTPUT] Add ifdefs around MEM_OUT handling in outmem (diff) | |
| download | dash-7bb413255368e94395237d789f522891093c5774.tar.gz dash-7bb413255368e94395237d789f522891093c5774.zip | |
[BUILTIN] Remove unnecessary restoration of format string in printf
Currently we try to preserve the format string which is stored in argv after temporarily modifying it. This is unnecessary as it's only ever used once. This patch gets rid of it. Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
| -rw-r--r-- | ChangeLog | 1 | ||||
| -rw-r--r-- | src/bltin/printf.c | 1 |
2 files changed, 1 insertions, 1 deletions
diff --git a/ChangeLog b/ChangeLog index 47ba519..026b47c 100644 --- a/ChangeLog +++ b/ChangeLog @@ -5,6 +5,7 @@ * Replace open-coded flushall in preadbuffer. * Add likely tag in outmem. * Add ifdefs around MEM_OUT handling in outmem. + * Remove unnecessary restoration of format string in printf. 2014-10-13 Eric Blake <eblake@redhat.com> diff --git a/src/bltin/printf.c b/src/bltin/printf.c index 213025f..d1181e3 100644 --- a/src/bltin/printf.c +++ b/src/bltin/printf.c @@ -207,7 +207,6 @@ pc: if (print_escape_str(start, param, array, getstr())) goto out; - *fmt = 'b'; break; case 'c': { int p = getchr(); |