change addOption callback argument from bool to Dynamic

pkg/args/lib/args.dart

Index: pkg/args/lib/args.dart
diff --git a/pkg/args/lib/args.dart b/pkg/args/lib/args.dart
index b49bc1a3dace91a51a128c0bbccec251fc0d963f..b923606eca3a0270eaab16f3e7c86eca787c4003 100644
--- a/pkg/args/lib/args.dart
+++ b/pkg/args/lib/args.dart
@@ -239,14 +239,14 @@ class ArgParser {
    */
   void addOption(String name, [String abbr, String help, List<String> allowed,
       Map<String, String> allowedHelp, String defaultsTo,
-      void callback(bool value), bool allowMultiple = false]) {
+      void callback(Dynamic value), bool allowMultiple = false]) {

[Bob Nystrom, 2012/09/18 00:43:25]

I don't know if the style guide mentions this (it should), but if a parameter's
type is Dynamic, we just don't annotate it at all. So you can just remove the
type annotation here.

[butlermatt, 2012/09/18 13:13:50]

Done.

     _addOption(name, abbr, help, allowed, allowedHelp, defaultsTo,
         callback, isFlag: false, allowMultiple: allowMultiple);
   }
 
   void _addOption(String name, String abbr, String help, List<String> allowed,
       Map<String, String> allowedHelp, defaultsTo,
-      void callback(bool value), [bool isFlag, bool negatable = false,
+      void callback(Dynamic value), [bool isFlag, bool negatable = false,

[Bob Nystrom, 2012/09/18 00:43:25]

And here.

[butlermatt, 2012/09/18 13:13:50]

Done.

       bool allowMultiple = false]) {
     // Make sure the name isn't in use.
     if (_options.containsKey(name)) {