-
Notifications
You must be signed in to change notification settings - Fork 568
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[swift] Introduce a ProtoExtensible
protocol that all messages that have been extended conform to.
#2790
Conversation
33e8a5b
to
24c554b
Compare
57c808b
to
22b62cd
Compare
24c554b
to
4a8f08e
Compare
a1e324e
to
6129012
Compare
6129012
to
f90ebff
Compare
f90ebff
to
953f4ad
Compare
* See the License for the specific language governing permissions and | ||
* limitations under the License. | ||
*/ | ||
public protocol ProtoExtensible { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should probably have some DocC for why this exists 😉
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
… have been extended conform to.
953f4ad
to
9069cf1
Compare
val isExtensible: Boolean | ||
get() = extensionsList.isNotEmpty() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What does having extensions make it extensible?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the list of reservations for extending a message, no?
If there are reservations then you can do extend Foo
or whatever in proto land.
That's what this is referring to.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Got it, yes.
This is generally harmless and another tiny step in preparation of a moderate re-design on how Wire proto extensions are generated.