change addOption callback argument from bool to Dynamic

pkg/args/lib/args.dart

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 /**
  * This library lets you define parsers for parsing raw command-line arguments
  * into a set of options and values using [GNU][] and [POSIX][] style options.
  *
  * ## Defining options ##
  *
@@ skipping 221 matching lines @@
   }
 
   /**
    * Defines a value-taking option. Throws an [IllegalArgumentException] if:
    *
    * * There is already an option with name [name].
    * * There is already an option using abbreviation [abbr].
    */
   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)) {
       throw new IllegalArgumentException('Duplicate option "$name".');
     }
 
     // Make sure the abbreviation isn't too long or in use.
     if (abbr != null) {
       if (abbr.length > 1) {
         throw new IllegalArgumentException(
@@ skipping 511 matching lines @@
       allowedBuffer.add(allowed);
       if (allowed == option.defaultValue) {
         allowedBuffer.add(' (default)');
       }
       first = false;
     }
     allowedBuffer.add(']');
     return allowedBuffer.toString();
   }
 }