[gold][patch] Change how to specify options for the plugins
Ian Lance Taylor
iant@google.com
Fri Dec 5 05:28:00 GMT 2008
Rafael Espindola <espindola@google.com> writes:
> 2008-12-04 Rafael Avila de Espindola <espindola@google.com>
>
> * options.cc (General_options::parse_plugin_opt): New.
> (General_options::add_plugin): The argument now is just the filename.
> (General_options::add_plugin_option): New.
> * options.h (plugin_opt): New.
> (add_plugin): Change argument name.
> (add_plugin_option): New.
> * plugin.cc (Plugin::load): Don't parse the plugin option.
> * plugin.h (Plugin::Plugin): Rename argument. Init filename_.
> (Plugin::add_option): New.
> (Plugin::args_): Change type.
> (Plugin::filename_): New.
> (Plugin_manager::add_plugin_option): New.
> * testsuite/Makefile.am (plugin_test_1): Use new syntax.
> * testsuite/Makefile.in: Regenerate.
> +
> +//Parse --plugin-opt.
> +
Please add a space between "//" and "Parse".
> + DEFINE_special(plugin_opt, options::TWO_DASHES, '\0',
> + N_("pass an option to the plugin"), N_("OPTION"));
Please capitalize "pass".
This is OK with those changes.
Thanks.
Ian
More information about the Binutils
mailing list