util/options.cc |   29 ++++++++++++++++-------------
 1 file changed, 16 insertions(+), 13 deletions(-)

New commits:
commit 76d57331117be8e0c2d4a2aee8341969b62b6888
Merge: 89cbd4d 9ee176e
Author: Behdad Esfahbod <beh...@behdad.org>
Date:   Mon Apr 13 23:40:35 2015 -0700

    Merge pull request #86 from cpfair/hb-shape-output-help-improvement
    
    Improve hb-shape/hb-view's help text w.r.t. output options

commit 9ee176ee978e6668c4faf00028811ce70979122e
Author: Collin Fair <c...@cpfx.ca>
Date:   Sat Feb 14 09:59:44 2015 -0500

    Stop hb-shape docs leaking into hb-view

diff --git a/util/options.cc b/util/options.cc
index fb74b1a..5d03caa 100644
--- a/util/options.cc
+++ b/util/options.cc
@@ -444,9 +444,7 @@ output_options_t::add_options (option_parser_t *parser)
   else
   {
     char *items = g_strjoinv ("/", const_cast<char **> (supported_formats));
-    text = g_strdup_printf ("Set output serialization format\n\n    Supported 
output formats are: %s\n%s", items,
-      "      text: [<glyph name or index>=<glyph cluster index within 
input>@<horizontal displacement>,<vertical displacement>+<horizontal 
advance>,<vertical advance>|...]\n"
-      "      json: [{\"g\": <glyph name or index>, \"ax\": <horizontal 
advance>, \"ay\": <vertical advance>, \"dx\": <horizontal displacement>, 
\"dy\": <vertical displacement>, \"cl\": <glyph cluster index within input>}, 
...]");
+    text = g_strdup_printf ("Set output format\n\n    Supported output formats 
are: %s", items);
     g_free (items);
     parser->free_later ((char *) text);
   }
@@ -459,7 +457,7 @@ output_options_t::add_options (option_parser_t *parser)
   };
   parser->add_group (entries,
                     "output",
-                    "Output detination & format options:",
+                    "Output destination & format options:",
                     "Options controlling the destination and form of the 
output",
                     this);
 }
@@ -706,9 +704,12 @@ format_options_t::add_options (option_parser_t *parser)
     {NULL}
   };
   parser->add_group (entries,
-                    "output-content",
-                    "Output content options (refer to --help-output for output 
syntax):",
-                    "Options controlling the content of the output",
+                    "output-syntax",
+                    "Output syntax:\n"
+         "    text: [<glyph name or index>=<glyph cluster index within 
input>@<horizontal displacement>,<vertical displacement>+<horizontal 
advance>,<vertical advance>|...]\n"
+         "    json: [{\"g\": <glyph name or index>, \"ax\": <horizontal 
advance>, \"ay\": <vertical advance>, \"dx\": <horizontal displacement>, 
\"dy\": <vertical displacement>, \"cl\": <glyph cluster index within input>}, 
...]\n"
+         "\nOutput syntax options:",
+                    "Options controlling the syntax of the output",
                     this);
 }
 
commit 952b8dbdf61da9b4814e09ad3c1b9b76483bef5c
Author: Collin Fair <c...@cpfx.ca>
Date:   Sat Feb 14 09:44:00 2015 -0500

    'All' of the above, not 'each'

diff --git a/util/options.cc b/util/options.cc
index 799ac4f..fb74b1a 100644
--- a/util/options.cc
+++ b/util/options.cc
@@ -699,10 +699,10 @@ format_options_t::add_options (option_parser_t *parser)
     {"show-text", 0, 0, G_OPTION_ARG_NONE, &this->show_text, "Prefix each line 
of output with its corresponding input text", NULL},
     {"show-unicode", 0, 0, G_OPTION_ARG_NONE, &this->show_unicode, "Prefix 
each line of output with its corresponding input codepoint(s)", NULL},
     {"show-line-num", 0, 0, G_OPTION_ARG_NONE, &this->show_line_num, "Prefix 
each line of output with its corresponding input line number", NULL},
-    {"verbose", 0, G_OPTION_FLAG_NO_ARG, G_OPTION_ARG_CALLBACK, (gpointer) 
&parse_verbose, "Prefix each line of output with each of the above", NULL},
+    {"verbose", 0, G_OPTION_FLAG_NO_ARG, G_OPTION_ARG_CALLBACK, (gpointer) 
&parse_verbose, "Prefix each line of output with all of the above", NULL},
     {"no-glyph-names", 0, G_OPTION_FLAG_REVERSE, G_OPTION_ARG_NONE, 
&this->show_glyph_names, "Output glyph indices instead of names", NULL},
     {"no-positions", 0, G_OPTION_FLAG_REVERSE, G_OPTION_ARG_NONE, 
&this->show_positions, "Do not output glyph positions", NULL},
-    {"no-clusters",    0, G_OPTION_FLAG_REVERSE, G_OPTION_ARG_NONE, 
&this->show_clusters, "Do not output cluster indices", NULL},
+    {"no-clusters", 0, G_OPTION_FLAG_REVERSE, G_OPTION_ARG_NONE, 
&this->show_clusters, "Do not output cluster indices", NULL},
     {NULL}
   };
   parser->add_group (entries,
commit 9e867b64467c46eea5cc9b1d5a9404a1221bbbfc
Author: Collin Fair <c...@cpfx.ca>
Date:   Sat Feb 14 09:32:04 2015 -0500

    Remove reference to --help-output-content in --help-output (as 
--help-output-format's options aren't available in hb-view)

diff --git a/util/options.cc b/util/options.cc
index c5b619d..799ac4f 100644
--- a/util/options.cc
+++ b/util/options.cc
@@ -459,7 +459,7 @@ output_options_t::add_options (option_parser_t *parser)
   };
   parser->add_group (entries,
                     "output",
-                    "Output detination & format options (refer to 
--help-output-content for content options):",
+                    "Output detination & format options:",
                     "Options controlling the destination and form of the 
output",
                     this);
 }
commit 1d55ffeb66381889f11ebb9039ca4ec2ca5efbab
Author: Collin Fair <c...@cpfx.ca>
Date:   Sat Feb 14 09:29:35 2015 -0500

    Add serialization syntax documentation. Clarify naming and wording in 
--help-output/--help-format

diff --git a/util/options.cc b/util/options.cc
index 7387a56..c5b619d 100644
--- a/util/options.cc
+++ b/util/options.cc
@@ -440,11 +440,13 @@ output_options_t::add_options (option_parser_t *parser)
   const char *text;
 
   if (NULL == supported_formats)
-    text = "Set output format";
+    text = "Set output serialization format";
   else
   {
     char *items = g_strjoinv ("/", const_cast<char **> (supported_formats));
-    text = g_strdup_printf ("Set output format\n\n    Supported output formats 
are: %s", items);
+    text = g_strdup_printf ("Set output serialization format\n\n    Supported 
output formats are: %s\n%s", items,
+      "      text: [<glyph name or index>=<glyph cluster index within 
input>@<horizontal displacement>,<vertical displacement>+<horizontal 
advance>,<vertical advance>|...]\n"
+      "      json: [{\"g\": <glyph name or index>, \"ax\": <horizontal 
advance>, \"ay\": <vertical advance>, \"dx\": <horizontal displacement>, 
\"dy\": <vertical displacement>, \"cl\": <glyph cluster index within input>}, 
...]");
     g_free (items);
     parser->free_later ((char *) text);
   }
@@ -457,8 +459,8 @@ output_options_t::add_options (option_parser_t *parser)
   };
   parser->add_group (entries,
                     "output",
-                    "Output options:",
-                    "Options controlling the output",
+                    "Output detination & format options (refer to 
--help-output-content for content options):",
+                    "Options controlling the destination and form of the 
output",
                     this);
 }
 
@@ -694,19 +696,19 @@ format_options_t::add_options (option_parser_t *parser)
 {
   GOptionEntry entries[] =
   {
-    {"no-glyph-names", 0, G_OPTION_FLAG_REVERSE, G_OPTION_ARG_NONE,    
&this->show_glyph_names,        "Use glyph indices instead of names",   NULL},
-    {"no-positions",   0, G_OPTION_FLAG_REVERSE, G_OPTION_ARG_NONE,    
&this->show_positions,          "Do not show glyph positions",          NULL},
-    {"no-clusters",    0, G_OPTION_FLAG_REVERSE, G_OPTION_ARG_NONE,    
&this->show_clusters,           "Do not show cluster mapping",          NULL},
-    {"show-text",      0, 0,                     G_OPTION_ARG_NONE,    
&this->show_text,               "Show input text",                      NULL},
-    {"show-unicode",   0, 0,                     G_OPTION_ARG_NONE,    
&this->show_unicode,            "Show input Unicode codepoints",        NULL},
-    {"show-line-num",  0, 0,                     G_OPTION_ARG_NONE,    
&this->show_line_num,           "Show line numbers",                    NULL},
-    {"verbose",                0, G_OPTION_FLAG_NO_ARG,  
G_OPTION_ARG_CALLBACK,(gpointer) &parse_verbose,      "Show everything",        
              NULL},
+    {"show-text", 0, 0, G_OPTION_ARG_NONE, &this->show_text, "Prefix each line 
of output with its corresponding input text", NULL},
+    {"show-unicode", 0, 0, G_OPTION_ARG_NONE, &this->show_unicode, "Prefix 
each line of output with its corresponding input codepoint(s)", NULL},
+    {"show-line-num", 0, 0, G_OPTION_ARG_NONE, &this->show_line_num, "Prefix 
each line of output with its corresponding input line number", NULL},
+    {"verbose", 0, G_OPTION_FLAG_NO_ARG, G_OPTION_ARG_CALLBACK, (gpointer) 
&parse_verbose, "Prefix each line of output with each of the above", NULL},
+    {"no-glyph-names", 0, G_OPTION_FLAG_REVERSE, G_OPTION_ARG_NONE, 
&this->show_glyph_names, "Output glyph indices instead of names", NULL},
+    {"no-positions", 0, G_OPTION_FLAG_REVERSE, G_OPTION_ARG_NONE, 
&this->show_positions, "Do not output glyph positions", NULL},
+    {"no-clusters",    0, G_OPTION_FLAG_REVERSE, G_OPTION_ARG_NONE, 
&this->show_clusters, "Do not output cluster indices", NULL},
     {NULL}
   };
   parser->add_group (entries,
-                    "format",
-                    "Format options:",
-                    "Options controlling the formatting of buffer contents",
+                    "output-content",
+                    "Output content options (refer to --help-output for output 
syntax):",
+                    "Options controlling the content of the output",
                     this);
 }
 
_______________________________________________
HarfBuzz mailing list
HarfBuzz@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/harfbuzz

Reply via email to