Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(939)

Side by Side Diff: test/protos/dart_options.proto

Issue 2043913005: Change how to set the Dart name of a field (Closed) Base URL: git@github.com:dart-lang/dart-protoc-plugin.git@master
Patch Set: better error checking, clean up pubspec Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« test/names_test.dart ('K') | « test/protos/dart_name.proto ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Experimental options controlling Dart code generation. 1 // Experimental options controlling Dart code generation.
2 syntax = "proto2"; 2 syntax = "proto2";
3 3
4 package dart_options; 4 package dart_options;
5 5
6 // This file must be modified for google internal use, 6 // This file must be modified for google internal use,
7 // because custom options only work when the package name 7 // because custom options only work when the package name
8 // agrees with the version of protoc we are using. 8 // agrees with the version of protoc we are using.
9 // (The import statement and "google.protobuf." prefix need to be changed.) 9 // (The import statement and "google.protobuf." prefix need to be changed.)
10 10
11 import "descriptor_2_5_opensource.proto"; 11 import "descriptor_2_5_opensource.proto";
12 12
13 extend google.protobuf.FileOptions { 13 extend google.protobuf.FileOptions {
14 14
15 // Applies the named mixin to all messages in this file. 15 // Applies the named mixin to all messages in this file.
16 // (May be overridden by the "mixin" option on a message.) 16 // (May be overridden by the "mixin" option on a message.)
17 // For now, "PbMapMixin" is the only available mixin. 17 // For now, "PbMapMixin" is the only available mixin.
18 optional string default_mixin = 96128839; 18 optional string default_mixin = 96128839;
19 } 19 }
20 20
21 extend google.protobuf.MessageOptions { 21 extend google.protobuf.MessageOptions {
22 22
23 // Applies the named mixin. 23 // Applies the named mixin.
24 // For now, "PbMapMixin" is the only available mixin. 24 // For now, "PbMapMixin" is the only available mixin.
25 // The empty string can be used to turn off mixins for this message. 25 // The empty string can be used to turn off mixins for this message.
26 optional string mixin = 96128839; 26 optional string mixin = 96128839;
27 } 27 }
28
29 extend google.protobuf.FieldOptions {
30
31 // Uses the given name for getters, setters and as suffixes for has/clear
32 // methods in the generated Dart file. Should be lowerCamelCase.
33 optional string dart_name = 28700919;
34 }
OLDNEW
« test/names_test.dart ('K') | « test/protos/dart_name.proto ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698