[PHP-dev 1378] mbstring.cのtypo

View: New views
2 Messages — Rating Filter:   Alert me  

[PHP-dev 1378] mbstring.cのtypo

by TAKAGI Masahiro-4 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

高木です。

# あえて bugs.php.net に投げるほどのネタでもなさそうなので、こちらに報告
# しておきます。

php-usersでのmb_strimwidthについての議論を眺めながらmbstringのソースコー
ドを追っていたら、ちょっとしたtypoを見つけてしまいました。

実際の挙動には何らかかわりのないところなので、何かのついでにでも修正して
おいていただけないでしょうか?

[HEAD]
Index: ext/mbstring/mbstring.c
===================================================================
RCS file: /repository/php-src/ext/mbstring/mbstring.c,v
retrieving revision 1.270
diff -u -r1.270 mbstring.c
--- ext/mbstring/mbstring.c 12 Jul 2007 15:28:37 -0000 1.270
+++ ext/mbstring/mbstring.c 2 Aug 2007 07:25:40 -0000
@@ -2316,7 +2316,7 @@
  convert_to_long_ex(arg2);
  from = Z_LVAL_PP(arg2);
  if (from < 0 || from > Z_STRLEN_PP(arg1)) {
- php_error_docref(NULL TSRMLS_CC, E_WARNING, "Start position is out
of reange");
+ php_error_docref(NULL TSRMLS_CC, E_WARNING, "Start position is out
of range");
  RETURN_FALSE;
  }

[PHP_5_2]
Index: ext/mbstring/mbstring.c
===================================================================
RCS file: /repository/php-src/ext/mbstring/mbstring.c,v
retrieving revision 1.224.2.22.2.24
diff -u -r1.224.2.22.2.24 mbstring.c
--- ext/mbstring/mbstring.c 12 Jul 2007 15:31:54 -0000 1.224.2.22.2.24
+++ ext/mbstring/mbstring.c 2 Aug 2007 07:25:51 -0000
@@ -2318,7 +2318,7 @@
  convert_to_long_ex(arg2);
  from = Z_LVAL_PP(arg2);
  if (from < 0 || from > Z_STRLEN_PP(arg1)) {
- php_error_docref(NULL TSRMLS_CC, E_WARNING, "Start position is out
of reange");
+ php_error_docref(NULL TSRMLS_CC, E_WARNING, "Start position is out
of range");
  RETURN_FALSE;
  }

[PHP_4_4]
Index: ext/mbstring/mbstring.c
===================================================================
RCS file: /repository/php-src/ext/mbstring/mbstring.c,v
retrieving revision 1.142.2.47.2.21
diff -u -r1.142.2.47.2.21 mbstring.c
--- ext/mbstring/mbstring.c 4 Apr 2007 15:28:18 -0000 1.142.2.47.2.21
+++ ext/mbstring/mbstring.c 2 Aug 2007 07:25:56 -0000
@@ -2618,7 +2618,7 @@
  convert_to_long_ex(arg2);
  from = Z_LVAL_PP(arg2);
  if (from < 0 || from > Z_STRLEN_PP(arg1)) {
- php_error_docref(NULL TSRMLS_CC, E_WARNING, "Start position is out
of reange");
+ php_error_docref(NULL TSRMLS_CC, E_WARNING, "Start position is out
of range");
  RETURN_FALSE;
  }


--
TAKAGI Masahiro mailto:matakagi@...
_______________________________________________
PHP-dev mailing list
PHP-dev@...
http://ml.php.gr.jp/mailman/listinfo/php-dev

[PHP-dev 1379] Re: mbstring.cのtypo

by Seiji Masugata :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

こんにちわ、桝形です。

> php-usersでのmb_strimwidthについての議論を眺めながらmbstringのソースコー
> ドを追っていたら、ちょっとしたtypoを見つけてしまいました。
>
> 実際の挙動には何らかかわりのないところなので、何かのついでにでも修正して
> おいていただけないでしょうか?

うは。お恥ずかしい限りです。
(最近)放置気味で恐縮ですが、折を見てコミットしておきます。

どうもありがとうございました。


ぼそっ。高木さんが翻訳した本、気になるなぁ〜
Extending And Embedding PHPもついでに。。。(^^;

--
Seiji Masugata <s.masugata@...>

_______________________________________________
PHP-dev mailing list
PHP-dev@...
http://ml.php.gr.jp/mailman/listinfo/php-dev
LightInTheBox - Buy quality products at wholesale price!