mirrors/phpv8
mirrors/phpv8
0
0
Fork 0
mirror of https://github.com/phpv8/v8js.git synced 2024-12-31 10:01:51 +00:00
Code Releases Activity

Merge pull request #299 from stesie/fix-compiler-warnings

Browse Source 
Fix compiler warnings
This commit is contained in:
Stefan Siegl 2017-03-11 18:13:39 +01:00 committed by GitHub
commit 86068afeca
7 changed files with 31 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

31
v8js_class.cc
View File

@ -654,7 +654,7 @@ static void v8js_compile_script(zval *this_ptr, const zend_string *str, const ze
return;
}
static void v8js_execute_script(zval *this_ptr, v8js_script *res, long flags, long time_limit, long memory_limit, zval **return_value)
static void v8js_execute_script(zval *this_ptr, v8js_script *res, long flags, long time_limit, size_t memory_limit, zval **return_value)
{
v8js_ctx *c = Z_V8JS_CTX_OBJ_P(this_ptr);
@ -702,13 +702,19 @@ static PHP_METHOD(V8Js, executeString)
return;
}
if (memory_limit < 0) {
zend_throw_exception(php_ce_v8js_exception,
"memory_limit must not be negative", 0);
return;
}
v8js_compile_script(getThis(), str, identifier, &res);
if (!res) {
RETURN_FALSE;
}
zend_try {
v8js_execute_script(getThis(), res, flags, time_limit, memory_limit, &return_value);
v8js_execute_script(getThis(), res, flags, time_limit, static_cast<size_t>(memory_limit), &return_value);
v8js_script_free(res);
}
zend_catch {
@ -757,11 +763,17 @@ static PHP_METHOD(V8Js, executeScript)
return;
}
if (memory_limit < 0) {
zend_throw_exception(php_ce_v8js_exception,
"memory_limit must not be negative", 0);
return;
}
if((res = (v8js_script *)zend_fetch_resource(Z_RES_P(zscript), PHP_V8JS_SCRIPT_RES_NAME, le_v8js_script)) == NULL) {
RETURN_FALSE;
}
v8js_execute_script(getThis(), res, flags, time_limit, memory_limit, &return_value);
v8js_execute_script(getThis(), res, flags, time_limit, static_cast<size_t>(memory_limit), &return_value);
}
/* }}} */
@ -907,14 +919,20 @@ static PHP_METHOD(V8Js, setMemoryLimit)
return;
}
if (memory_limit < 0) {
zend_throw_exception(php_ce_v8js_exception,
"memory_limit must not be negative", 0);
return;
}
c = Z_V8JS_CTX_OBJ_P(getThis());
c->memory_limit = memory_limit;
c->memory_limit = static_cast<size_t>(memory_limit);
V8JSG(timer_mutex).lock();
for (std::deque< v8js_timer_ctx* >::iterator it = V8JSG(timer_stack).begin();
it != V8JSG(timer_stack).end(); it ++) {
if((*it)->ctx == c && !(*it)->killed) {
(*it)->memory_limit = memory_limit;
(*it)->memory_limit = static_cast<size_t>(memory_limit);
}
}
V8JSG(timer_mutex).unlock();
@ -1076,7 +1094,6 @@ static PHP_METHOD(V8Js, registerExtension)
static PHP_METHOD(V8Js, getExtensions)
{
v8js_jsext *jsext;
zend_ulong index;
zend_string *key;
zval *val, ext;
@ -1091,7 +1108,7 @@ static PHP_METHOD(V8Js, getExtensions)
#endif
if (v8js_process_globals.extensions) {
ZEND_HASH_FOREACH_KEY_VAL(v8js_process_globals.extensions, index, key, val) {
ZEND_HASH_FOREACH_STR_KEY_VAL(v8js_process_globals.extensions, key, val) {
if (key) {
jsext = (v8js_jsext *) Z_PTR_P(val);
array_init(&ext);

2
v8js_class.h
View File

@ -44,7 +44,7 @@ struct v8js_ctx {
long time_limit;
bool time_limit_hit;
long memory_limit;
size_t memory_limit;
bool memory_limit_hit;
long average_object_size;

4
v8js_object_export.cc
View File

@ -379,9 +379,7 @@ static void v8js_named_property_enumerator(const v8::PropertyCallbackInfo<v8::Ar
ZVAL_OBJ(&tmp_zv, object);
proptable = Z_OBJPROP(tmp_zv);
zval *data;
ZEND_HASH_FOREACH_KEY_VAL(proptable, index, key, data) {
ZEND_HASH_FOREACH_KEY(proptable, index, key) {
if(key) {
/* skip protected and private members */
if(ZSTR_VAL(key)[0] == '\0') {

2
v8js_timer.cc
View File

@ -124,7 +124,7 @@ void v8js_timer_thread(zend_v8js_globals *globals) /* {{{ */
/* }}} */
void v8js_timer_push(long time_limit, long memory_limit, v8js_ctx *c) /* {{{ */
void v8js_timer_push(long time_limit, size_t memory_limit, v8js_ctx *c) /* {{{ */
{
V8JSG(timer_mutex).lock();

4
v8js_timer.h
View File

@ -18,14 +18,14 @@
struct v8js_timer_ctx
{
long time_limit;
long memory_limit;
size_t memory_limit;
std::chrono::time_point<std::chrono::high_resolution_clock> time_point;
v8js_ctx *ctx;
bool killed;
};
void v8js_timer_thread(zend_v8js_globals *globals);
void v8js_timer_push(long time_limit, long memory_limit, v8js_ctx *c);
void v8js_timer_push(long time_limit, size_t memory_limit, v8js_ctx *c);
#endif /* V8JS_TIMER_H */

2
v8js_v8.cc
View File

@ -95,7 +95,7 @@ void v8js_v8_init() /* {{{ */
* heap allocated memory).
*/
void v8js_v8_call(v8js_ctx *c, zval **return_value,
long flags, long time_limit, long memory_limit,
long flags, long time_limit, size_t memory_limit,
std::function< v8::Local<v8::Value>(v8::Isolate *) >& v8_call) /* {{{ */
{
char *tz = NULL;

2
v8js_v8.h
View File

@ -53,7 +53,7 @@ static inline const char * ToCString(const v8::String::Utf8Value &value) /* {{{
void v8js_v8_init();
void v8js_v8_call(v8js_ctx *c, zval **return_value,
long flags, long time_limit, long memory_limit,
long flags, long time_limit, size_t memory_limit,
std::function< v8::Local<v8::Value>(v8::Isolate *) >& v8_call);
void v8js_terminate_execution(v8::Isolate *isolate);