From 3782fa20edcd3e4bdbbfa5ffbd430449b97c009a Mon Sep 17 00:00:00 2001
From: wm4 <wm4@nowhere>
Date: Sun, 22 Dec 2013 23:04:19 +0100
Subject: path: change mp_splitext() semantics

Including the "." in the returned extension was too inconvenient. I
think originally, the semantics were supposed to work like in Python,
but screw this.

Also, return NULL instead of "" on failure (which is what its only user
actually seems to expect).
---
 player/screenshot.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'player')

diff --git a/player/screenshot.c b/player/screenshot.c
index 7336c64e97..500960aee4 100644
--- a/player/screenshot.c
+++ b/player/screenshot.c
@@ -347,7 +347,7 @@ void screenshot_to_file(struct MPContext *mpctx, const char *filename, int mode,
     }
     char *ext = mp_splitext(filename, NULL);
     if (ext)
-        opts.format = ext + 1; // omit '.'
+        opts.format = ext;
     struct mp_image *image = screenshot_get(mpctx, mode);
     if (!image) {
         screenshot_msg(ctx, SMSG_ERR, "Taking screenshot failed.");
-- 
cgit v1.2.3