Index: ext/pgsql/pgsql.c =================================================================== RCS file: /repository/php-src/ext/pgsql/pgsql.c,v retrieving revision 1.244.2.19 diff -u -r1.244.2.19 pgsql.c --- ext/pgsql/pgsql.c 22 Jun 2003 16:48:16 -0000 1.244.2.19 +++ ext/pgsql/pgsql.c 6 Nov 2004 22:34:59 -0000 @@ -3446,7 +3446,7 @@ switch (Z_TYPE_PP(val)) { case IS_STRING: if (Z_STRLEN_PP(val) == 0) { - ZVAL_STRING(new_val, "NULL", 1); + ZVAL_NULL(new_val); } else { if (!strcmp(Z_STRVAL_PP(val), "t") || !strcmp(Z_STRVAL_PP(val), "T") || @@ -3454,14 +3454,14 @@ !strcmp(Z_STRVAL_PP(val), "true") || !strcmp(Z_STRVAL_PP(val), "True") || !strcmp(Z_STRVAL_PP(val), "yes") || !strcmp(Z_STRVAL_PP(val), "Yes") || !strcmp(Z_STRVAL_PP(val), "1")) { - ZVAL_STRING(new_val, "'t'", 1); + ZVAL_TRUE(new_val); } else if (!strcmp(Z_STRVAL_PP(val), "f") || !strcmp(Z_STRVAL_PP(val), "F") || !strcmp(Z_STRVAL_PP(val), "n") || !strcmp(Z_STRVAL_PP(val), "N") || !strcmp(Z_STRVAL_PP(val), "false") || !strcmp(Z_STRVAL_PP(val), "False") || !strcmp(Z_STRVAL_PP(val), "no") || !strcmp(Z_STRVAL_PP(val), "No") || !strcmp(Z_STRVAL_PP(val), "0")) { - ZVAL_STRING(new_val, "'f'", 1); + ZVAL_FALSE(new_val); } else { php_error_docref(NULL TSRMLS_CC, E_NOTICE, "Detected invalid value (%s) for PostgreSQL %s field (%s).", Z_STRVAL_PP(val), Z_STRVAL_PP(type), field); @@ -3473,15 +3473,16 @@ case IS_LONG: case IS_BOOL: if (Z_LVAL_PP(val)) { - ZVAL_STRING(new_val, "'t'", 1); + ZVAL_TRUE(new_val); } else { - ZVAL_STRING(new_val, "'f'", 1); + ZVAL_FALSE(new_val); } break; case IS_NULL: - ZVAL_STRING(new_val, "NULL", 1); + ZVAL_NULL(new_val); + break; default: @@ -3500,7 +3501,7 @@ switch (Z_TYPE_PP(val)) { case IS_STRING: if (Z_STRLEN_PP(val) == 0) { - ZVAL_STRING(new_val, "NULL", 1); + ZVAL_NULL(new_val); } else { /* FIXME: better regex must be used */ @@ -3523,7 +3524,7 @@ break; case IS_NULL: - ZVAL_STRING(new_val, "NULL", 1); + ZVAL_NULL(new_val); break; default: @@ -3542,7 +3543,7 @@ switch (Z_TYPE_PP(val)) { case IS_STRING: if (Z_STRLEN_PP(val) == 0) { - ZVAL_STRING(new_val, "NULL", 1); + ZVAL_NULL(new_val); } else { /* FIXME: better regex must be used */ @@ -3564,7 +3565,7 @@ break; case IS_NULL: - ZVAL_STRING(new_val, "NULL", 1); + ZVAL_NULL(new_val); break; default: @@ -3583,7 +3584,7 @@ case IS_STRING: if (Z_STRLEN_PP(val) == 0) { if (opt & PGSQL_CONV_FORCE_NULL) { - ZVAL_STRING(new_val, "NULL", 1); + ZVAL_NULL(new_val); } else { ZVAL_STRING(new_val, empty_string, 1); @@ -3616,7 +3617,7 @@ break; case IS_NULL: - ZVAL_STRING(new_val, "NULL", 1); + ZVAL_NULL(new_val); break; default: @@ -3634,7 +3635,7 @@ switch (Z_TYPE_PP(val)) { case IS_STRING: if (Z_STRLEN_PP(val) == 0) { - ZVAL_STRING(new_val, "NULL", 1); + ZVAL_NULL(new_val); } else { /* FIXME: Better regex must be used */ @@ -3658,7 +3659,7 @@ break; case IS_NULL: - ZVAL_STRING(new_val, "NULL", 1); + ZVAL_NULL(new_val); break; default: @@ -3675,7 +3676,7 @@ switch (Z_TYPE_PP(val)) { case IS_STRING: if (Z_STRLEN_PP(val) == 0) { - ZVAL_STRING(new_val, "NULL", 1); + ZVAL_NULL(new_val); } else { /* FIXME: Better regex must be used */ @@ -3690,7 +3691,7 @@ break; case IS_NULL: - ZVAL_STRING(new_val, "NULL", 1); + ZVAL_NULL(new_val); break; default: @@ -3708,7 +3709,7 @@ switch(Z_TYPE_PP(val)) { case IS_STRING: if (Z_STRLEN_PP(val) == 0) { - ZVAL_STRING(new_val, "NULL", 1); + ZVAL_NULL(new_val); } else { /* FIXME: better regex must be used */ @@ -3723,7 +3724,7 @@ break; case IS_NULL: - ZVAL_STRING(new_val, "NULL", 1); + ZVAL_NULL(new_val); break; default: @@ -3739,7 +3740,7 @@ switch(Z_TYPE_PP(val)) { case IS_STRING: if (Z_STRLEN_PP(val) == 0) { - ZVAL_STRING(new_val, "NULL", 1); + ZVAL_NULL(new_val); } else { /* FIXME: better regex must be used */ @@ -3754,7 +3755,7 @@ break; case IS_NULL: - ZVAL_STRING(new_val, "NULL", 1); + ZVAL_NULL(new_val); break; default: @@ -3770,7 +3771,7 @@ switch(Z_TYPE_PP(val)) { case IS_STRING: if (Z_STRLEN_PP(val) == 0) { - ZVAL_STRING(new_val, "NULL", 1); + ZVAL_NULL(new_val); } else { /* FIXME: better regex must be used */ @@ -3785,7 +3786,7 @@ break; case IS_NULL: - ZVAL_STRING(new_val, "NULL", 1); + ZVAL_NULL(new_val); break; default: @@ -3801,7 +3802,7 @@ switch(Z_TYPE_PP(val)) { case IS_STRING: if (Z_STRLEN_PP(val) == 0) { - ZVAL_STRING(new_val, "NULL", 1); + ZVAL_NULL(new_val); } else { /* FIXME: better regex must be used */ @@ -3817,7 +3818,7 @@ break; case IS_NULL: - ZVAL_STRING(new_val, "NULL", 1); + ZVAL_NULL(new_val); break; default: @@ -3833,7 +3834,7 @@ switch (Z_TYPE_PP(val)) { case IS_STRING: if (Z_STRLEN_PP(val) == 0) { - ZVAL_STRING(new_val, "NULL", 1); + ZVAL_NULL(new_val); } else { unsigned char *tmp; @@ -3860,7 +3861,7 @@ break; case IS_NULL: - ZVAL_STRING(new_val, "NULL", 1); + ZVAL_NULL(new_val); break; default: @@ -3877,7 +3878,7 @@ switch(Z_TYPE_PP(val)) { case IS_STRING: if (Z_STRLEN_PP(val) == 0) { - ZVAL_STRING(new_val, "NULL", 1); + ZVAL_NULL(new_val); } else { if (php_pgsql_convert_match(Z_STRVAL_PP(val), "^([0-9a-f]{2,2}:){5,5}[0-9a-f]{2,2}$", 1 TSRMLS_CC) == FAILURE) { @@ -3891,7 +3892,7 @@ break; case IS_NULL: - ZVAL_STRING(new_val, "NULL", 1); + ZVAL_NULL(new_val); break; default: