apply changes from code review

Author: LamentXU123

NEWS

@@ -121,7 +121,7 @@ PHP                                                                        NEWS
   . Fixed bug GH-19926 (reset internal pointer earlier while splicing array
     while COW violation flag is still set). (alexandre-daubois)
   . Added form feed (\f) in the default trimmed characters of trim(), rtrim()
-    and ltrim(). (Weilin Du, LamentXU)
+    and ltrim(). (Weilin Du)
   . Invalid mode values now throw in array_filter() instead of being silently
     defaulted to 0. (Jorg Sowa)
   . Fixed bug GH-21058 (error_log() crashes with message_type 3 and

ext/standard/tests/strings/trim.phpt

@@ -18,6 +18,9 @@ var_dump("ABC" === trim("ABC\x50\xC1\x60\x90","\x50..\xC1"));
 var_dump("ABC\x50\xC1" === trim("ABC\x50\xC1\x60\x90","\x51..\xC0"));
 var_dump("ABC\x50" === trim("ABC\x50\xC1\x60\x90","\x51..\xC1"));
 var_dump("ABC" === trim("ABC\x50\xC1\x60\x90","\x50..\xC1"));
+var_dump('ABC' ===  trim('\fABC'));
+var_dump('ABC' === ltrim('\fABC'));
+var_dump('ABC' === rtrim('\fABC'));
 
 ?>
 --EXPECT--
@@ -36,3 +39,6 @@ bool(true)
 bool(true)
 bool(true)
 bool(true)
+bool(true)
+bool(true)
+bool(true)

tests/classes/tostring_001.phpt

@@ -24,16 +24,6 @@ class test3
         return [];
     }
 }
-
-class test4
-{
-    function __toString()
-    {
-        echo __METHOD__ . "()\n";
-        return "Converted\f";
-    }
-}
-
 echo "====test1====\n";
 $o = new test1;
 print_r($o);
@@ -87,9 +77,7 @@ try {
 } catch (Error $e) {
     echo $e->getMessage(), "\n";
 }
-echo "====test11====\n";
-$o = new test4();
-var_dump(trim($o));
+
 ?>
 ====DONE====
 --EXPECT--
@@ -144,7 +132,4 @@ object(test3)#2 (0) {
 }
 test3::__toString()
 test3::__toString(): Return value must be of type string, array returned
-====test11====
-test4::__toString()
-string(9) "Converted"
-====DONE====
+====DONE====