diff options
| author | wm4 <wm4@nowhere> | 2014-09-26 13:52:55 +0200 |
|---|---|---|
| committer | wm4 <wm4@nowhere> | 2014-09-26 13:52:55 +0200 |
| commit | 8fd954ac8e7fe5d2770d74c3780bdd5f44aa0906 (patch) | |
| tree | 446d999d5d62ec69f9348ad26d67de6be837493f /player | |
| parent | fdf40743bcf99d6ce5cc258e59fec30e0cb203e8 (diff) | |
| download | mpv-8fd954ac8e7fe5d2770d74c3780bdd5f44aa0906.tar.bz2 mpv-8fd954ac8e7fe5d2770d74c3780bdd5f44aa0906.tar.xz | |
build: add -Wno-format-zero-length
This warning makes absolutely no sense. Passing an empty string to
printf-like functions is perfectly fine. In the OSD case, it just sets
an empty message, practically clearing the OSD.
Diffstat (limited to 'player')
| -rw-r--r-- | player/command.c | 2 | ||||
| -rw-r--r-- | player/osd.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/player/command.c b/player/command.c index c37e45faec..0399905e23 100644 --- a/player/command.c +++ b/player/command.c @@ -3698,7 +3698,7 @@ int run_command(MPContext *mpctx, mp_cmd_t *cmd) if (opts->osd_level > 0) { set_osd_msg(mpctx, osdl, osd_duration, "OSD level: %d", opts->osd_level); } else { - set_osd_msg(mpctx, 0, 0, "%s", ""); + set_osd_msg(mpctx, 0, 0, ""); } break; } diff --git a/player/osd.c b/player/osd.c index f1495a21a9..15cceab95f 100644 --- a/player/osd.c +++ b/player/osd.c @@ -422,7 +422,7 @@ static void add_seek_osd_messages(struct MPContext *mpctx) if (mpctx->add_osd_seek_info & OSD_SEEK_INFO_TEXT) { // Never in term-osd mode if (mpctx->video_out && mpctx->opts->term_osd != 1) { - if (set_osd_msg(mpctx, 1, mpctx->opts->osd_duration, "%s", "")) + if (set_osd_msg(mpctx, 1, mpctx->opts->osd_duration, "")) mpctx->osd_show_pos = true; } } |