From 4466aeea1d801caa2ffc777db7e7f10bf726cd1b Mon Sep 17 00:00:00 2001 From: Ian Barber Date: Fri, 23 Nov 2012 19:06:06 +0000 Subject: [PATCH] Fix win and mac compiler warnings Verified with latest build of libsvm (3.14). Tweaked some types to avoid a bunch of smallish compiler warnings. --- package.xml | 11 ++++++----- svm.c | 20 ++++++++++---------- 2 files changed, 16 insertions(+), 15 deletions(-) diff --git a/package.xml b/package.xml index 4ecd9a2..c8ac987 100644 --- a/package.xml +++ b/package.xml @@ -14,10 +14,10 @@ http://pear.php.net/dtd/package-2.0.xsd"> ian.barber@gmail.co.uk yes - 2012-09-16 - + 2012-11-23 + - 0.1.8 + 0.1.9 0.1.5 @@ -26,8 +26,9 @@ http://pear.php.net/dtd/package-2.0.xsd"> PHP - - Fixed GH#2 issue when loading data from a resource - - Modify include search path to start more generic + - Verified works with 3.14 + - Fixed option return format + - Removed unused variable diff --git a/svm.c b/svm.c index 5f6469a..4ae4cc0 100644 --- a/svm.c +++ b/svm.c @@ -290,7 +290,7 @@ static struct svm_problem* php_svm_read_array(php_svm_object *intern, php_svm_mo char *endptr; int i, num_labels, elements; int j = 0, max_index = 0, inst_max_index = 0; - long index; + unsigned long index; struct svm_problem *problem; /* If reading multiple times make sure that we don't leak */ @@ -439,7 +439,7 @@ Take an incoming parameter and convert it into a PHP array of svmlight style dat */ static zval* php_svm_get_data_from_param(php_svm_object *intern, zval *zparam TSRMLS_DC) { - zval *data, *return_value; + zval *data; zend_bool our_stream = 0; zend_bool need_read = 1; php_stream *stream = NULL; @@ -597,13 +597,13 @@ PHP_METHOD(svm, getOptions) add_index_long(return_value, phpsvm_probability, intern->param.probability == 1 ? TRUE : FALSE); add_index_long(return_value, phpsvm_shrinking, intern->param.shrinking == 1 ? TRUE : FALSE); - add_index_long(return_value, phpsvm_gamma, intern->param.gamma); - add_index_long(return_value, phpsvm_coef0, intern->param.coef0); - add_index_long(return_value, phpsvm_nu, intern->param.nu); - add_index_long(return_value, phpsvm_cache_size, intern->param.cache_size); - add_index_long(return_value, phpsvm_C, intern->param.C); - add_index_long(return_value, phpsvm_eps, intern->param.eps); - add_index_long(return_value, phpsvm_p, intern->param.p); + add_index_double(return_value, phpsvm_gamma, intern->param.gamma); + add_index_double(return_value, phpsvm_coef0, intern->param.coef0); + add_index_double(return_value, phpsvm_nu, intern->param.nu); + add_index_double(return_value, phpsvm_cache_size, intern->param.cache_size); + add_index_double(return_value, phpsvm_C, intern->param.C); + add_index_double(return_value, phpsvm_eps, intern->param.eps); + add_index_double(return_value, phpsvm_p, intern->param.p); } /* }}} */ @@ -779,7 +779,7 @@ PHP_METHOD(svm, train) HashTable *weights_ht; int i; char *key; - long index; + unsigned long index; zend_bool status = 0; weights = 0;