[vlc-devel] [PATCH] FreeType: mark some options as safe

Rémi Denis-Courmont remi at remlab.net
Fri Nov 5 18:24:41 CET 2010


Font is not safe because it uses the file system, and is dependent on
the local system anyway. YUVP is only for encoding, so it seems
irrelevant. Other options seem fine.
---
 modules/misc/freetype.c |    6 ++++++
 1 files changed, 6 insertions(+), 0 deletions(-)

diff --git a/modules/misc/freetype.c b/modules/misc/freetype.c
index a325981..9348267 100644
--- a/modules/misc/freetype.c
+++ b/modules/misc/freetype.c
@@ -151,22 +151,28 @@ vlc_module_begin ()
 
     add_integer( "freetype-fontsize", 0, FONTSIZE_TEXT,
                  FONTSIZE_LONGTEXT, true )
+        change_safe()
 
     /* opacity valid on 0..255, with default 255 = fully opaque */
     add_integer_with_range( "freetype-opacity", 255, 0, 255, NULL,
         OPACITY_TEXT, OPACITY_LONGTEXT, true )
+        change_safe()
 
     /* hook to the color values list, with default 0x00ffffff = white */
     add_integer( "freetype-color", 0x00FFFFFF, COLOR_TEXT,
                  COLOR_LONGTEXT, false )
         change_integer_list( pi_color_values, ppsz_color_descriptions )
+        change_safe()
 
     add_integer( "freetype-rel-fontsize", 16, FONTSIZER_TEXT,
                  FONTSIZER_LONGTEXT, false )
         change_integer_list( pi_sizes, ppsz_sizes_text )
+        change_safe()
+
     add_integer( "freetype-effect", 2, EFFECT_TEXT,
                  EFFECT_LONGTEXT, false )
         change_integer_list( pi_effects, ppsz_effects_text )
+        change_safe()
 
     add_bool( "freetype-yuvp", false, YUVP_TEXT,
               YUVP_LONGTEXT, true )
-- 
1.7.2.3




More information about the vlc-devel mailing list