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.

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

Optimize/annotate kernel/object.h #2003

Merged
merged 4 commits into from Feb 10, 2014
Merged

Optimize/annotate kernel/object.h #2003

merged 4 commits into from Feb 10, 2014

Conversation

ghost
Copy link

@ghost ghost commented Feb 9, 2014

  • Remove not used code
  • Add const where appropriate
  • Annotate parameters as not NULL where appropriate to help static code analyzers find possible bugs
  • Move helper functions to object.h and make them inline

* Fix crashes under valgrind when Zend MM is used;
* Double the number of preallocated frames (based on the longest
  execution path)
* More checks for the debug mode
phalcon pushed a commit that referenced this pull request Feb 10, 2014
Optimize/annotate kernel/object.h
@phalcon phalcon merged commit dd1e4a6 into phalcon:1.3.0 Feb 10, 2014
@ghost ghost deleted the obj-opt branch February 11, 2014 12:00
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.

None yet

3 participants