Parameterize uri resolution and parsing of options, use package:path

lib/code_generator.dart

 // Copyright (c) 2013, 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.
 
 part of protoc;
 
 abstract class ProtobufContainer {
   String get package;
   String get classname;
   String get fqname;
   String get packageImportPrefix => package.replaceAll('.', r'$');
 }
 
 class CodeGenerator extends ProtobufContainer {
   final Stream<List<int>> _streamIn;
   final IOSink _streamOut;
   final IOSink _streamErr;
 
   CodeGenerator(this._streamIn, this._streamOut, this._streamErr);
 
-  void generate() {
+  /// Runs the code generator. The optional [optionsParser] can be used to
+  /// change how command line options are parsed (see [GenerationOptionsParser]
+  /// for details), and [outputConfiguration] can be used to override where
+  /// generated files are created and how imports between generated files are
+  /// constructed (see [OutputConfiguration] for details).
+  void generate({
+      GenerationOptionsParser optionsParser: parseDefaultGenerationOptions,
+      OutputConfiguration outputConfiguration}) {
     _streamIn
         .fold(new BytesBuilder(), (builder, data) => builder..add(data))
         .then((builder) => builder.takeBytes())
         .then((List<int> bytes) {
             var request = new CodeGeneratorRequest.fromBuffer(bytes);
             var response = new CodeGeneratorResponse();
 
             // Parse the options in the request. Return the errors is any.
-            var options = new GenerationOptions(request, response);
+            var options = optionsParser(request, response);
             if (options == null) {
               _streamOut.add(response.writeToBuffer());
               return;
             }
 
-            var ctx = new GenerationContext(options);
+            var ctx = new GenerationContext(options, outputConfiguration == null
+                ? new DefaultOutputConfiguration() : outputConfiguration);
             List<FileGenerator> generators = <FileGenerator>[];
             for (FileDescriptorProto file in request.protoFile) {
               var generator = new FileGenerator(file, this, ctx);
               if (request.fileToGenerate.contains(file.name)) {
                 generators.add(generator);
               }
             }
 
             response.file.addAll(
                 generators.map((filegen) => filegen.generateResponse()));
             _streamOut.add(response.writeToBuffer());
         });
   }
 
   String get package => '';
   String get classname => null;
   String get fqname => '';
 }
-
-
-class GenerationOptions {
-  final Map<String, String> fieldNameOptions;
-
-  GenerationOptions._(this.fieldNameOptions);
-
-  // Parse the options in the request. If there was an error in the
-  // options null is returned and the error is set on the response.
-  factory GenerationOptions(request, response) {

[Siggi Cherem (dart-lang), 2014/05/07 23:30:44]

I refactored this a bit in options.dart, the main change is that I split the
parsing of the option list, from the parsing of an individual option. That makes
it more configurable.

-    var fieldNameOptions = <String, String>{};
-    var parameter = request.parameter != null ? request.parameter : '';
-    List<String> options = parameter.trim().split(',');
-    List<String> errors = [];
-    for (var option in options) {
-      option = option.trim();
-      if (option.isEmpty) continue;
-      List<String> nameValue = option.split('=');
-      if (nameValue.length != 1 && nameValue.length != 2) {
-        errors.add('Illegal option: $option');
-        continue;
-      }
-      String name = nameValue[0].trim();
-      String value;
-      if (nameValue.length > 1) value = nameValue[1].trim();
-      if (name == 'field_name') {
-        if (value == null) {
-          errors.add('Illegal option: $option');
-          continue;
-        }
-        List<String> fromTo = value.split('|');
-        if (fromTo.length != 2) {
-          errors.add('Illegal option: $option');
-          continue;
-        }
-        var fromName = fromTo[0].trim();
-        var toName = fromTo[1].trim();
-        if (fromName.length == 0 || toName.length == 0) {
-          errors.add('Illegal option: $option');
-          continue;
-        }
-        fieldNameOptions['.$fromName'] = toName;
-      } else {
-        errors.add('Illegal option: $option');
-      }
-    }
-    if (errors.length > 0) {
-      response.error = errors.join('\n');
-      return null;
-    } else {
-      return new GenerationOptions._(fieldNameOptions);
-    }
-  }
-
-  String fieldNameOption(String fqname) => fieldNameOptions[fqname];
-}