diff options
author | Niclas Zeising <zeising@daemonic.se> | 2017-09-26 21:34:56 +0200 |
---|---|---|
committer | Niclas Zeising <zeising@daemonic.se> | 2017-09-26 21:42:30 +0200 |
commit | f49f0879e651d8c92468f232febe143db2cbcd36 (patch) | |
tree | 10c81c14bf0cebd5607b48d841d98bf0900612ff /src/collage.c | |
parent | 9b4341e12c6be32b654478cdf3a4e8a631c4e31f (diff) |
Fix dead code
The check if buffer == NULL is always false, since buffer is an
autoamtic variable allocated when entering the function. What we
instead want to do is to check if the string is empty after the call to
exif_get_info(), since that means we could not read any exif
information.
When the code once more is enabled, I discovered that we need to copy
the information string into info_buf as well as into buffer, since it
is the former that is used to print the exif information on top of the
picture. Without this change, imlib warns about trying to write NULL
strings.
Diffstat (limited to 'src/collage.c')
0 files changed, 0 insertions, 0 deletions