Skip to content

Conversation

scottanderson
Copy link

This should serve to further reduce the amount of overhead associated with this library.

@scottanderson
Copy link
Author

This makes the parts of the API that get extended more clear in regards to what behavior is required (abstract), and what behavior can not be overridden (final). In any application that does not override handle*, this is a trivial change.

@loopj
Copy link
Collaborator

loopj commented Mar 27, 2012

I'd prefer to keep these methods overridable, since it allows for more flexibility in applications.

@loopj loopj closed this Mar 27, 2012
@scottanderson
Copy link
Author

Can you provide a little more detail on which methods, specifically, you intend to preserve, and what the reasoning is? The handler behavior isn't something I see a reason to override - is this the concern? The rest of the commit should be innocuous, and if needed I'd like to pick out the offensive parts so the rest can be merged.

@korkag korkag mentioned this pull request Sep 8, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants