[chromium-dev] Re: Does anyone have objections to adding a GetSwitchValues() accessor to CommandLine?
On Wed, Jun 3, 2009 at 12:06 AM, Book'em Dano daniel.c...@gmail.com wrote: Does anyone have objections to including such a function? It would just return a copy of std::mapstd::string, StringType switches_; I'd like to add such a function so that I can iterate over the command line args (both switches and loose values) and validate that only expected values are present. How do you decide what is unexpected? The current system allows any part of the code to define and use switches. TVL -D --~--~-~--~~~---~--~~ Chromium Developers mailing list: chromium-dev@googlegroups.com View archives, change email options, or unsubscribe: http://groups.google.com/group/chromium-dev -~--~~~~--~~--~--~---
[chromium-dev] Re: Does anyone have objections to adding a GetSwitchValues() accessor to CommandLine?
For Chromium that may be the case, but there are other projects that are also using the CommandLine class that may/do have other reqs. On Jun 3, 5:02 am, Thomas Van Lenten thoma...@chromium.org wrote: On Wed, Jun 3, 2009 at 12:06 AM, Book'em Dano daniel.c...@gmail.com wrote: Does anyone have objections to including such a function? It would just return a copy of std::mapstd::string, StringType switches_; I'd like to add such a function so that I can iterate over the command line args (both switches and loose values) and validate that only expected values are present. How do you decide what is unexpected? The current system allows any part of the code to define and use switches. TVL -D --~--~-~--~~~---~--~~ Chromium Developers mailing list: chromium-dev@googlegroups.com View archives, change email options, or unsubscribe: http://groups.google.com/group/chromium-dev -~--~~~~--~~--~--~---
[chromium-dev] Re: Does anyone have objections to adding a GetSwitchValues() accessor to CommandLine?
Would an iterator method on CommandLine serve the same purpose? It seems like that could work without having to make a copy... --Amanda On Wed, Jun 3, 2009 at 12:06 AM, Book'em Dano daniel.c...@gmail.com wrote: Does anyone have objections to including such a function? It would just return a copy of std::mapstd::string, StringType switches_; I'd like to add such a function so that I can iterate over the command line args (both switches and loose values) and validate that only expected values are present. -D --~--~-~--~~~---~--~~ Chromium Developers mailing list: chromium-dev@googlegroups.com View archives, change email options, or unsubscribe: http://groups.google.com/group/chromium-dev -~--~~~~--~~--~--~---
[chromium-dev] Re: Does anyone have objections to adding a GetSwitchValues() accessor to CommandLine?
Sure. that woudl work too. I'm kinda curious why this was removed in the first place. If i recall correctly, we used to have such a method. On Jun 2, 9:09 pm, Amanda Walker ama...@chromium.org wrote: Would an iterator method on CommandLine serve the same purpose? It seems like that could work without having to make a copy... --Amanda On Wed, Jun 3, 2009 at 12:06 AM, Book'em Dano daniel.c...@gmail.com wrote: Does anyone have objections to including such a function? It would just return a copy of std::mapstd::string, StringType switches_; I'd like to add such a function so that I can iterate over the command line args (both switches and loose values) and validate that only expected values are present. -D --~--~-~--~~~---~--~~ Chromium Developers mailing list: chromium-dev@googlegroups.com View archives, change email options, or unsubscribe: http://groups.google.com/group/chromium-dev -~--~~~~--~~--~--~---