Attachment 135230 Details for Bug 103536 – Patch to use UTF-8 filenames on Windows

[patch] Patch to use UTF-8 filenames on Windows

cairo-utf8-filenames.patch (text/plain), 4.67 KB, created by Tom Schoonjans on 2017-11-03 16:53:08 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Tom Schoonjans

Created: 2017-11-03 16:53:08 UTC

Size: 4.67 KB

patch

obsolete

>From 2513930ce5ae699ed568ac2cd3eb2ad71af1d0a2 Mon Sep 17 00:00:00 2001
>From: Tom Schoonjans <Tom.Schoonjans@diamond.ac.uk>
>Date: Thu, 2 Nov 2017 16:55:22 +0000
>Subject: [PATCH] Use UTF-8 filenames on Windows
>
>Until now fopen was used on Windows to open files for reading and
>writing. This assumed however that the filename would be encoded in the
>current codepage, which is a major inconvenience and makes it even
>impossible to use filenames that use characters from more than one
>codepage. This patch enforces the use of UTF-8 filenames on all
>platforms.
>
>Based on the work of Owen Taylor (https://lists.cairographics.org/archives/cairo/2007-February/009591.html)
>---
> src/cairo-output-stream.c | 52 ++++++++++++++++++++++++++++++++++++++++++++++-
> src/cairo-png.c           | 10 +++++----
> src/cairoint.h            |  4 ++++
> 3 files changed, 61 insertions(+), 5 deletions(-)
>
>diff --git a/src/cairo-output-stream.c b/src/cairo-output-stream.c
>index f43f212ed..123142787 100644
>--- a/src/cairo-output-stream.c
>+++ b/src/cairo-output-stream.c
>@@ -661,7 +661,7 @@ _cairo_output_stream_create_for_filename (const char *filename)
>     if (filename == NULL)
> 	return _cairo_null_stream_create ();
> 
>-    file = fopen (filename, "wb");
>+    file = _cairo_fopen (filename, "wb");
>     if (file == NULL) {
> 	switch (errno) {
> 	case ENOMEM:
>@@ -804,3 +804,53 @@ _cairo_null_stream_create (void)
> 
>     return stream;
> }
>+
>+/**
>+ * _cairo_fopen:
>+ * @filename: filename to open
>+ * @mode: mode string with which to open the file
>+ * 
>+ * Exactly like the C library function, but possibly doing encoding
>+ * conversion on the filename. On Unix platforms, the filename is
>+ * passed directly to the system, but on Windows, the filename is
>+ * interpreted as UTF-8, rather than in a codepage that would depend
>+ * on system settings.
>+ * 
>+ * Return value: The newly opened file, or NULL if an error occured.
>+ **/
>+FILE *
>+_cairo_fopen (const char *filename, const char *mode)
>+{
>+#ifdef _WIN32 /* also defined on x86_64 */
>+    uint16_t *filename_w;
>+    uint16_t *mode_w;
>+    FILE *result;
>+
>+    if (filename == NULL || mode == NULL) {
>+	errno = EINVAL;
>+	return NULL;
>+    }
>+    
>+    if (_cairo_utf8_to_utf16 (filename, -1, &filename_w, NULL) != CAIRO_STATUS_SUCCESS) {
>+	errno = EINVAL;
>+	return NULL;
>+    }
>+
>+    if (_cairo_utf8_to_utf16 (mode, -1, &mode_w, NULL) != CAIRO_STATUS_SUCCESS) {
>+	free (filename_w);
>+	errno = EINVAL;
>+	return NULL;
>+    }
>+
>+    result = _wfopen(filename_w, mode_w);
>+
>+    free (filename_w);
>+    free (mode_w);
>+
>+    return result;
>+    
>+#else /* Use fopen directly */
>+    return fopen (filename, mode);
>+#endif  
>+}
>+
>diff --git a/src/cairo-png.c b/src/cairo-png.c
>index eab2537bc..ff0af8829 100644
>--- a/src/cairo-png.c
>+++ b/src/cairo-png.c
>@@ -345,7 +345,8 @@ stdio_write_func (png_structp png, png_bytep data, png_size_t size)
> /**
>  * cairo_surface_write_to_png:
>  * @surface: a #cairo_surface_t with pixel contents
>- * @filename: the name of a file to write to
>+ * @filename: the name of a file to write to; on Windows this filename
>+ *   is encoded in UTF-8.
>  *
>  * Writes the contents of @surface to a new file @filename as a PNG
>  * image.
>@@ -373,7 +374,7 @@ cairo_surface_write_to_png (cairo_surface_t	*surface,
>     if (surface->finished)
> 	return _cairo_error (CAIRO_STATUS_SURFACE_FINISHED);
> 
>-    fp = fopen (filename, "wb");
>+    fp = _cairo_fopen (filename, "wb");
>     if (fp == NULL) {
> 	switch (errno) {
> 	case ENOMEM:
>@@ -738,7 +739,8 @@ read_png (struct png_read_closure_t *png_closure)
> 
> /**
>  * cairo_image_surface_create_from_png:
>- * @filename: name of PNG file to load
>+ * @filename: name of PNG file to load. On Windows this filename
>+ *   is encoded in UTF-8.
>  *
>  * Creates a new image surface and initializes the contents to the
>  * given PNG file.
>@@ -765,7 +767,7 @@ cairo_image_surface_create_from_png (const char *filename)
>     struct png_read_closure_t png_closure;
>     cairo_surface_t *surface;
> 
>-    png_closure.closure = fopen (filename, "rb");
>+    png_closure.closure = _cairo_fopen (filename, "rb");
>     if (png_closure.closure == NULL) {
> 	cairo_status_t status;
> 	switch (errno) {
>diff --git a/src/cairoint.h b/src/cairoint.h
>index 6f8303c3c..68ae75f07 100644
>--- a/src/cairoint.h
>+++ b/src/cairoint.h
>@@ -1919,6 +1919,10 @@ _cairo_matrix_multiply (cairo_matrix_t *r,
> cairo_private void
> _cairo_observers_notify (cairo_list_t *observers, void *arg);
> 
>+/* Open a file with a UTF-8 filename */
>+cairo_private FILE *
>+_cairo_fopen (const char *filename, const char *mode);
>+
> /* Avoid unnecessary PLT entries.  */
> slim_hidden_proto (cairo_clip_preserve);
> slim_hidden_proto (cairo_close_path);
>-- 
>2.14.2
>
>n_proto (cairo_clip_preserve);
> slim_hidden_proto (cairo_close_path);
>-- 
>2.14.2
>

Actions: View | Splinter Review

Attachments on bug 103536: 135220 | 135230 | 135258

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.