Skip to content
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.

Sign up for GitHub

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

Jump to bottom

public class ExtensionRegistry becomes non-final in 2.6.0; by design? #22

Closed
protobufel opened this issue Sep 15, 2014 · 2 comments
Closed

public class ExtensionRegistry becomes non-final in 2.6.0; by design? #22

protobufel opened this issue Sep 15, 2014 · 2 comments

Comments

@protobufel
Copy link

Is it a bug, or a feature?

David
@liujisi
Copy link
Contributor

liujisi commented Sep 15, 2014

Internally we have a subclass in the same package which is not opensourced. The ExtensionRegistry is not intended to be sub-classed by the users though.

@protobufel
Copy link
Author

I think it's a potential security risk; version 2.5.0 had this class final. Just raising question, if it is an oversight.

@liujisi liujisi closed this as completed Mar 5, 2015
@jakiechris jakiechris mentioned this issue Nov 20, 2017
Closed
@gatkinso gatkinso mentioned this issue Nov 23, 2017
Closed
@catskul catskul mentioned this issue Aug 21, 2018
Closed
TeBoring pushed a commit to TeBoring/protobuf that referenced this issue Jan 19, 2019
@haberman
Merge pull request protocolbuffers#22 from martinmaly/upbc
e6dddd6 
Generating enums and intentries only if the tables are non-empty.
@arnow117 arnow117 mentioned this issue Apr 1, 2019
Closed
GerHobbelt pushed a commit to GerHobbelt/protobuf that referenced this issue Aug 4, 2024
@qjk-redlands
Merge pull request protocolbuffers#22 from Esri/quinten/runtimecore
a9e194c 
zlib-ng project file changes
yordis pushed a commit to yordis/protobuf that referenced this issue Dec 8, 2024
@tony612
Merge pull request protocolbuffers#22 from ElaWorkshop/wire-types-macro
0e358c2 
Use macro to centralize wire types declaration
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@liujisi @protobufel