Re: cvs: php4 /ext/bcmath bcmath.c

I agree with him. We have standards and people have worked hard to indent the source code in a consistent manner (especially people like Jani). It doesn't take more time to indent nicely. Andi At 06:54 PM 4/2/2003 -0800, Rasmus Lerdorf wrote:

Oh, give me a break Jani. A space here or there makes absolutely no difference to the readability of the code, but all these whitespace commits get in the way of any real work when trying to track down problems via cvs diffs. On Thu, 3 Apr 2003, Jani Taskinen wrote:

sniper                Wed Apr  2 20:16:34 2003 EDT

Modified files:

    /php4/ext/bcmath  bcmath.c

Log: Another CS fix. # Rasmus, at least try keep the style consistent.. Index: php4/ext/bcmath/bcmath.c diff -u php4/ext/bcmath/bcmath.c:1.53 php4/ext/bcmath/bcmath.c:1.54

--- php4/ext/bcmath/bcmath.c:1.53     Wed Apr  2 18:51:52 2003

+++ php4/ext/bcmath/bcmath.c Wed Apr 2 20:16:34 2003 @@ -16,7 +16,7 @@ +----------------------------------------------------------------------+ */ -/* $Id: bcmath.c,v 1.53 2003/04/02 23:51:52 rasmus Exp $ */ +/* $Id: bcmath.c,v 1.54 2003/04/03 01:16:34 sniper Exp $ */ #ifdef HAVE_CONFIG_H #include "config.h" @@ -97,7 +97,11 @@

      if (cfg_get_long("bcmath.scale", &bc_precision) == FAILURE) {
              bc_precision = 0;
      }
-     if(bc_precision<0) bc_precision=0;

+

+     if (bc_precision < 0) {
+             bc_precision = 0;
+     }

+

      bc_init_numbers(TSRMLS_C);

      return SUCCESS;

@@ -145,7 +149,7 @@

                                      WRONG_PARAM_COUNT;
                              }
                              convert_to_long_ex(scale_param);
-                             scale = (int) (Z_LVAL_PP(scale_param)<0) ? 0:Z_LVAL_PP(scale_param);
+                             scale = (int) (Z_LVAL_PP(scale_param) < 0) ? 0 : Z_LVAL_PP(scale_param);
                              break;
              default:
                              WRONG_PARAM_COUNT;

@@ -191,7 +195,7 @@

                                      WRONG_PARAM_COUNT;
                              }
                              convert_to_long_ex(scale_param);
-                             scale = (int) (Z_LVAL_PP(scale_param)<0) ? 0:Z_LVAL_PP(scale_param);
+                             scale = (int) (Z_LVAL_PP(scale_param) < 0) ? 0 : Z_LVAL_PP(scale_param);
                              break;
              default:
                              WRONG_PARAM_COUNT;

@@ -237,7 +241,7 @@

                                      WRONG_PARAM_COUNT;
                              }
                              convert_to_long_ex(scale_param);
-                             scale = (int) (Z_LVAL_PP(scale_param)<0) ? 0:Z_LVAL_PP(scale_param);
+                             scale = (int) (Z_LVAL_PP(scale_param) < 0) ? 0 : Z_LVAL_PP(scale_param);
                              break;
              default:
                              WRONG_PARAM_COUNT;

@@ -283,7 +287,7 @@

                                      WRONG_PARAM_COUNT;
                              }
                              convert_to_long_ex(scale_param);
-                             scale = (int) (Z_LVAL_PP(scale_param)<0) ? 0:Z_LVAL_PP(scale_param);
+                             scale = (int) (Z_LVAL_PP(scale_param) < 0) ? 0 : Z_LVAL_PP(scale_param);
                              break;
              default:
                              WRONG_PARAM_COUNT;

@@ -296,7 +300,7 @@

      bc_init_num(&result TSRMLS_CC);
      php_str2num(&first, Z_STRVAL_PP(left) TSRMLS_CC);
      php_str2num(&second, Z_STRVAL_PP(right) TSRMLS_CC);
-     switch (bc_divide (first, second, &result, scale TSRMLS_CC)) {
+     switch (bc_divide(first, second, &result, scale TSRMLS_CC)) {
              case 0: /* OK */
                      if (result->n_scale > scale) {
                              result->n_scale = scale;

@@ -411,7 +415,7 @@

                                      WRONG_PARAM_COUNT;
                              }
                              convert_to_long_ex(scale_param);
-                             scale = (int) (Z_LVAL_PP(scale_param)<0) ? 0:Z_LVAL_PP(scale_param);
+                             scale = (int) (Z_LVAL_PP(scale_param) < 0) ? 0 : Z_LVAL_PP(scale_param);
                              break;
              default:
                              WRONG_PARAM_COUNT;

@@ -457,7 +461,7 @@

                                      WRONG_PARAM_COUNT;
                              }
                              convert_to_long_ex(scale_param);
-                             scale = (int) (Z_LVAL_PP(scale_param)<0) ? 0:Z_LVAL_PP(scale_param);
+                             scale = (int) (Z_LVAL_PP(scale_param) < 0) ? 0 : Z_LVAL_PP(scale_param);
                              break;
              default:
                              WRONG_PARAM_COUNT;

@@ -500,7 +504,7 @@

                                      WRONG_PARAM_COUNT;
                              }
                              convert_to_long_ex(scale_param);
-                             scale = (int) (Z_LVAL_PP(scale_param)<0) ? 0:Z_LVAL_PP(scale_param);
+                             scale = (int) (Z_LVAL_PP(scale_param) < 0) ? 0 : Z_LVAL_PP(scale_param);
                              break;
              default:
                              WRONG_PARAM_COUNT;

@@ -534,7 +538,7 @@

      }

      convert_to_long_ex(new_scale);
-     bc_precision = (Z_LVAL_PP(new_scale)<0) ? 0 : Z_LVAL_PP(new_scale);
+     bc_precision = (Z_LVAL_PP(new_scale) < 0) ? 0 : Z_LVAL_PP(new_scale);

      RETURN_TRUE;

} -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php