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

kernel/fcall.c optimizations #843

Merged
merged 23 commits into from Jul 15, 2013
Merged

kernel/fcall.c optimizations #843

Changes from 1 commit
Commits
Show all changes
23 commits
Select commit Hold shift + click to select a range
80df871
Keep clang happy
sjinks Jul 14, 2013
48252bd
Keep the compiler happy
sjinks Jul 14, 2013
7ff15ad
Added __func__ macro for MSVC
sjinks Jul 14, 2013
367438b
Map PHALCON_ALLOC_ZVAL to ALLOC_INIT_ZVAL
sjinks Jul 14, 2013
dd11807
Keep the compiler happy
sjinks Jul 14, 2013
ad4edad
Get rid of noreturn parameter, make the code C99 compatible, remove d…
sjinks Jul 14, 2013
36eacbc
Removed more duplicated code
sjinks Jul 14, 2013
5f346f6
Bug fixes
sjinks Jul 14, 2013
a17f3b2
Bug fixes
sjinks Jul 14, 2013
e994a1a
More deduplication
sjinks Jul 14, 2013
4ce7ba8
Bug fixes
sjinks Jul 14, 2013
38a25d2
Added -Wall to ext/install
sjinks Jul 14, 2013
4e10498
Updated build/*/php_phalcon.h
sjinks Jul 14, 2013
b4c4e7e
Bug fixes in scripts/gen-build.php
sjinks Jul 14, 2013
95837fd
Bug fixes
sjinks Jul 14, 2013
ff759f1
Removed not used functions
sjinks Jul 14, 2013
1285268
Make sure the code in build/ works
sjinks Jul 14, 2013
4b96ccc
Added phalcon_call_func_params_w() and phalcon_call_method_params_w()…
sjinks Jul 14, 2013
1ece18b
Use _w calls in recursive Phalcon\Config methods
sjinks Jul 14, 2013
2de0cc7
More code deduplication
sjinks Jul 14, 2013
3a2206c
Even more code deduplication
sjinks Jul 14, 2013
a92bdca
Removed dead code
sjinks Jul 14, 2013
84469ae
Updated build/ [ci skip]
sjinks Jul 14, 2013
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
Keep clang happy
  • Loading branch information
@sjinks
sjinks committed Jul 14, 2013
commit 80df8715e94df20217c5dc1fa12293776735d2e1
19 changes: 7 additions & 12 deletions ext/kernel/fcall.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -66,9 +66,7 @@ static inline int phalcon_find_scope(zend_class_entry *ce, char *method_name, in
}

if (!lower) {
if (lcname) {
efree(lcname);
}
efree(lcname);
}

return FAILURE;
Expand All @@ -95,10 +93,7 @@ static inline int phalcon_find_parent_scope(zend_class_entry *ce, const char *ac
ce = ce->parent;
}

if (lcname) {
efree(lcname);
}

efree(lcname);
return FAILURE;
}

Expand Down Expand Up @@ -532,7 +527,7 @@ int phalcon_call_method_five_params(zval *return_value, zval *object, char *meth
/**
* Call single static function that requires an arbitrary number of parameters
*/
inline int phalcon_call_static_func_params(zval *return_value, char *class_name, int class_length, char *method_name, int method_length, zend_uint param_count, zval *params[], int noreturn TSRMLS_DC){
static inline int phalcon_call_static_func_params(zval *return_value, char *class_name, int class_length, char *method_name, int method_length, zend_uint param_count, zval *params[], int noreturn TSRMLS_DC){

zval *fn, *fn_class, *fn_method;
int status;
Expand Down Expand Up @@ -600,7 +595,7 @@ inline int phalcon_call_static_func_params(zval *return_value, char *class_name,
/**
* Call single static function which not requires parameters
*/
inline int phalcon_call_static_func(zval *return_value, char *class_name, int class_length, char *method_name, int method_length, int noreturn TSRMLS_DC){
static inline int phalcon_call_static_func(zval *return_value, char *class_name, int class_length, char *method_name, int method_length, int noreturn TSRMLS_DC){

zval *fn, *fn_class, *fn_method;
int status = FAILURE;
Expand Down Expand Up @@ -776,7 +771,7 @@ int phalcon_call_self_func(zval *return_value, zval *object, char *method_name,
/**
* Call self-class static function which requires parameters
*/
inline int phalcon_call_self_func_params(zval *return_value, zval *object, char *method_name, int method_len, zend_uint param_count, zval *params[], int noreturn TSRMLS_DC){
static inline int phalcon_call_self_func_params(zval *return_value, zval *object, char *method_name, int method_len, zend_uint param_count, zval *params[], int noreturn TSRMLS_DC){

int success;
zend_class_entry *ce, *active_scope = NULL;
Expand Down Expand Up @@ -936,7 +931,7 @@ int phalcon_call_static_zval_func(zval *return_value, zval *mixed_name, zval *me
/**
* Call single static function on a zval which requires parameters
*/
inline int phalcon_call_static_zval_func_params(zval *return_value, zval *mixed_name, zval *method, zend_uint param_count, zval *params[], int noreturn TSRMLS_DC){
static inline int phalcon_call_static_zval_func_params(zval *return_value, zval *mixed_name, zval *method, zend_uint param_count, zval *params[], int noreturn TSRMLS_DC){

zval *fn;
int status = FAILURE;
Expand Down Expand Up @@ -1007,7 +1002,7 @@ inline int phalcon_call_static_zval_func_params(zval *return_value, zval *mixed_
/**
* Call single static function on a zval which requires parameters
*/
inline int phalcon_call_static_zval_str_func_params(zval *return_value, zval *mixed_name, char *method_name, int method_len, zend_uint param_count, zval *params[], int noreturn TSRMLS_DC){
static inline int phalcon_call_static_zval_str_func_params(zval *return_value, zval *mixed_name, char *method_name, int method_len, zend_uint param_count, zval *params[], int noreturn TSRMLS_DC){

zval *fn;
int status = FAILURE;
Expand Down