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

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

Issue 2086253002: Allow application of external mixins to generated dart protos. (Closed) Base URL: https://github.com/dart-lang/dart-protoc-plugin.git@master
Patch Set: Add dart options for override annotations. These make it possible to annotate generated getters, se… Created 4 years, 5 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
« no previous file with comments | « lib/src/dart_options.pb.dart ('k') | test/protos/mixins.proto » ('j') | 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
(...skipping 41 matching lines...) Expand 10 before | Expand all | Expand 10 after
52 52
53 extend google.protobuf.MessageOptions { 53 extend google.protobuf.MessageOptions {
54 54
55 // Applies the named mixin. 55 // Applies the named mixin.
56 // For now, "PbMapMixin" is the only available mixin. 56 // For now, "PbMapMixin" is the only available mixin.
57 // The empty string can be used to turn off mixins for this message. 57 // The empty string can be used to turn off mixins for this message.
58 optional string mixin = 96128839; 58 optional string mixin = 96128839;
59 } 59 }
60 60
61 extend google.protobuf.FieldOptions { 61 extend google.protobuf.FieldOptions {
62 // Adds @override annotation to the field's getter (for use with mixins).
63 optional bool override_getter = 28205290;
64
65 // Adds @override annotation to the field's setter (for use with mixins).
66 optional bool override_setter = 28937366;
67
68 // Adds @override annotation to the field's hasX() method (for use with
69 // mixins).
70 optional bool override_has_method = 28937461;
71
72 // Adds @override annotation to the field's clearX() method (for use with
73 // mixins).
74 optional bool override_clear_method = 28907907;
62 75
63 // Uses the given name for getters, setters and as suffixes for has/clear 76 // Uses the given name for getters, setters and as suffixes for has/clear
64 // methods in the generated Dart file. Should be lowerCamelCase. 77 // methods in the generated Dart file. Should be lowerCamelCase.
65 optional string dart_name = 28700919; 78 optional string dart_name = 28700919;
66 } 79 }
OLDNEW
« no previous file with comments | « lib/src/dart_options.pb.dart ('k') | test/protos/mixins.proto » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698