Clean DONE tags from tests

Remove most of the `===DONE===` tags and its variations.
Keep `===DONE===` if the test output otherwise becomes empty.

Closes GH-4872.

Author: villfa

Zend/tests/019.phpt

@@ -360,7 +360,6 @@ var_dump($global_var);
 //Note: No error conditions relating to passing arguments can be tested
 // because these are not functions but statements, it will result in syntax error.
 ?>
-===DONE===
 --EXPECTF--
 *** Testing unset(), empty() & isset() with scalar variables ***
 -- Iteration 1 --
@@ -1330,4 +1329,3 @@ bool(false)
 bool(false)
 bool(true)
 int(10)
-===DONE===

Zend/tests/bug26166.phpt

@@ -61,11 +61,9 @@ try {
 }
 
 ?>
-===DONE===
 --EXPECT--
 Hello World!
 ===NONE===
 Method NoneTest::__toString() must return a string value
 ===THROW===
 This is an error!
-===DONE===

Zend/tests/bug26229.phpt

@@ -23,7 +23,5 @@ catch(Exception $e)
 	echo $e->getMessage() . "\n";
 }
 ?>
-===DONE===
 --EXPECT--
 Objects returned by array_iterator::getIterator() must be traversable or implement interface Iterator
-===DONE===

Zend/tests/bug26696.phpt

@@ -16,7 +16,5 @@ for ($i = 0; $i < $len; $i++) {
 }
 
 ?>
-===DONE===
 --EXPECT--
 a-s-d-d-/-?+
-===DONE===

Zend/tests/bug26697.phpt

@@ -12,10 +12,8 @@ spl_autoload_register(function ($name) {
 var_dump(class_exists('NotExistingClass'));
 
 ?>
-===DONE===
 --EXPECT--
 {closure}(NotExistingClass)
 bool(false)
 {closure}(NotExistingClass), done
 bool(false)
-===DONE===

Zend/tests/bug26698.phpt

@@ -65,9 +65,7 @@ $p->callOne();
 $p->callTwo();
 $p->callThree();
 ?>
-===DONE===
 --EXPECT--
 Caught: NONE
 Caught: NONE
 Caught: NONE
-===DONE===

Zend/tests/bug26802.phpt

@@ -30,8 +30,6 @@ foo::$method();
 
 
 ?>
-===DONE===
 --EXPECT--
 global_func
 foo::foo_func
-===DONE===

Zend/tests/bug27304.phpt

@@ -16,8 +16,6 @@ Staticexample::test();
 $b->test();
 
 ?>
-===DONE===
 --EXPECT--
 bool(false)
 bool(false)
-===DONE===

Zend/tests/bug27669.phpt

@@ -10,7 +10,5 @@ Bug #27669 (PHP 5 didn't support all possibilities for calling static methods dy
 	$y[0] = 'hello';
 	A::{$y[0]}();
 ?>
-===DONE===
 --EXPECTF--
 Hello World
-===DONE===

Zend/tests/bug27798.phpt

@@ -32,7 +32,6 @@ var_dump(get_object_vars(new Base));
 var_dump(get_object_vars(new Child));
 
 ?>
-===DONE===
 --EXPECT--
 Base::__construct
 array(3) {
@@ -69,4 +68,3 @@ array(1) {
   ["Foo"]=>
   int(1)
 }
-===DONE===

Zend/tests/bug28442.phpt

@@ -44,7 +44,6 @@ var_dump(ClassB::$prop);
 var_dump(ClassC::$prop);
 
 ?>
-===DONE===
 --EXPECT--
 ===INIT===
 string(1) "A"
@@ -62,4 +61,3 @@ string(2) "B2"
 string(2) "A2"
 string(2) "C2"
 string(2) "C2"
-===DONE===

Zend/tests/bug28444.phpt

@@ -52,7 +52,6 @@ var_dump($t->x = 5);
 var_dump($y->z->x = 6);
 
 ?>
-===DONE===
 --EXPECT--
 object(ObjectOne)#2 (1) {
   ["x"]=>
@@ -75,4 +74,3 @@ Overloaded::__get(z)
 int(5)
 Overloaded::__get(z)
 int(6)
-===DONE===

Zend/tests/bug29368.phpt

@@ -27,8 +27,6 @@ try
 unset($bar);
 
 ?>
-===DONE===
 --EXPECT--
 Foo::__construct
 Caught exception!
-===DONE===

Zend/tests/bug29505.phpt

@@ -11,7 +11,6 @@ class Test {
 var_dump(get_class_vars('Test'));
 
 ?>
-===DONE===
 --EXPECT--
 array(2) {
   ["empty"]=>
@@ -28,4 +27,3 @@ array(2) {
     }
   }
 }
-===DONE===

Zend/tests/bug30162.phpt

@@ -41,7 +41,6 @@ $db = new hariCow;
 
 var_dump($db);
 ?>
-===DONE===
 --EXPECTF--
 Warning: Undefined variable: db in %s on line %d
 NULL
@@ -51,4 +50,3 @@ object(hariCow)#%d (2) {
   ["y"]=>
   string(1) "y"
 }
-===DONE===

Zend/tests/bug30346.phpt

@@ -18,7 +18,5 @@ echo $post[$id.'_show'];
 echo "\n";
 
 ?>
-===DONE===
 --EXPECT--
 page_show
-===DONE===

Zend/tests/bug30407.phpt

@@ -11,8 +11,6 @@ function haricow($a = 'one') {
 haricow();
 haricow();
 ?>
-===DONE===
 --EXPECT--
 string(3) "one"
 string(3) "one"
-===DONE===

Zend/tests/bug30725.phpt

@@ -26,7 +26,5 @@ catch(Exception $e)
 }
 
 ?>
-===DONE===
 --EXPECT--
 Caught
-===DONE===

Zend/tests/bug30998.phpt

@@ -13,9 +13,7 @@ set_error_handler('my_error');
 
 $f = fopen("/tmp/blah", "r");
 ?>
-===DONE===
 --EXPECTF--
 fopen(/tmp/blah): failed to open stream: %s (2) in %s:%d
 
 Warning: fopen(/tmp/blah): failed to open stream: %s in %s on line %d
-===DONE===

Zend/tests/bug31102.phpt

@@ -36,7 +36,6 @@ while($test++ < 5)
 }
 ?>
 ===DONE===
-<?php exit(0); ?>
 --EXPECTF--
 {closure}(Test1,1)
 Caught: Test1::__construct

Zend/tests/bug31720.phpt

@@ -10,8 +10,6 @@ try {
     echo $e->getMessage(), "\n";
 }
 ?>
-===DONE===
 --EXPECTF--
 Warning: Undefined variable: nonesuchvar in %s on line %d
 array_walk() expects parameter 2 to be a valid callback, first array member is not a valid class name or object
-===DONE===

Zend/tests/bug32226.phpt

@@ -30,7 +30,5 @@ catch(Exception $e)
 }
 
 ?>
-===DONE===
 --EXPECT--
 Caught
-===DONE===

Zend/tests/bug32252.phpt

@@ -40,8 +40,6 @@ catch (Exception $e)
 }
 
 ?>
-===DONE===
 --EXPECT--
 Test::offsetSet(-1, 123)
 CAUGHT
-===DONE===

Zend/tests/bug32290.phpt

@@ -94,8 +94,6 @@ echo "\n===E===\n";
 var_dump($x->doSomethingStatic(1));
 
 ?>
-===DONE===
-<?php exit(0); ?>
 --EXPECT--
 ===A===
 TestB::doSomething(1)
@@ -121,4 +119,3 @@ int(1)
 TestB::doSomethingStatic(1)
 TestA::doSomethingStatic(2)
 int(1)
-===DONE===

Zend/tests/bug37212.phpt

@@ -38,7 +38,6 @@ $B->copyValue($C);
 var_dump($B);
 
 ?>
-===DONE===
 --EXPECTF--
 object(B)#%d (1) {
   ["value":protected]=>
@@ -52,4 +51,3 @@ object(B)#%d (1) {
   ["value":protected]=>
   string(1) "C"
 }
-===DONE===

Zend/tests/bug37667.phpt

@@ -26,7 +26,6 @@ $obj->property[] = 2;
 var_dump($obj);
 
 ?>
-===DONE===
 --EXPECTF--
 string(3) "bar"
 
@@ -50,4 +49,3 @@ object(Test)#%d (1) {
     string(3) "bar"
   }
 }
-===DONE===

Zend/tests/bug46196.phpt

@@ -22,10 +22,8 @@ function myErrorHandler($errno, $errstr, $errfile, $errline)
 }
 
 ?>
-===DONE===
 --EXPECT--
 *** Testing restore_error_handler() : error bug #46196 ***
 NULL
 bool(true)
 NULL
-===DONE===

Zend/tests/bug48215.phpt

@@ -27,10 +27,8 @@ class B extends A
 $b = new B();
 $b->A();
 ?>
-===DONE===
 --EXPECT--
 B::__construct
 A::__construct
 B::A
 A::A
-===DONE===

Zend/tests/bug66015.phpt

@@ -24,12 +24,10 @@ class Test
 $test = new Test();
 ?>
 
-===DONE===
 --EXPECT--
 array (
   1 => 'first',
   2 => 'second',
   3 => 'third',
   4 => 4,
 )
-===DONE===

Zend/tests/bug68797.phpt

@@ -7,7 +7,5 @@ precision=17
 
 echo 2.2250738585072012e-308, "\n";
 ?>
-==DONE==
 --EXPECT--
 2.2250738585072014E-308
-==DONE==

Zend/tests/bug71572.phpt

@@ -10,7 +10,6 @@ var_dump($str[3] = "");
 var_dump($str[10] = "");
 var_dump($str);
 ?>
-==DONE==
 --EXPECTF--
 Warning: Cannot assign an empty string to a string offset in %s on line %d
 NULL
@@ -24,4 +23,3 @@ NULL
 Warning: Cannot assign an empty string to a string offset in %s on line %d
 NULL
 string(3) "abc"
-==DONE==

Zend/tests/bug72918.phpt

@@ -13,10 +13,8 @@ $string[-3]
 
 EOT;
 ?>
-===DONE===
 --EXPECT--
 foo
 c
 foo
 c
-===DONE===

Zend/tests/bug76965.phpt

@@ -15,7 +15,6 @@ END;
 
 var_dump(parse_ini_string($ini, false, INI_SCANNER_RAW));
 ?>
-===DONE===
 --EXPECT--
 array(7) {
   [1]=>
@@ -33,4 +32,3 @@ array(7) {
   [7]=>
   string(7) "foo"bar"
 }
-===DONE===

Zend/tests/bug77291.phpt

@@ -35,8 +35,6 @@ var_dump(isset($foo->a));
 $foo = new Foo45;
 var_dump($foo->a);
 ?>
-===DONE===
 --EXPECT--
 bool(true)
 string(10) "Some value"
-===DONE===

Zend/tests/bug_debug_backtrace.phpt

@@ -18,11 +18,9 @@ eval("foo();");
 
 echo "Done\n";
 ?>
-===DONE===
 --EXPECTF--
 #0  boo() called at [%s:%d]
 #1  bar() called at [%s:%d]
 #2  foo() called at [%s(%d) : eval()'d code:1]
 #3  eval() called at [%s:%d]
 Done
-===DONE===

Zend/tests/closure_034.phpt

@@ -7,7 +7,6 @@ $a = function () use(&$a) {};
 var_dump($a);
 
 ?>
-===DONE===
 --EXPECTF--
 object(Closure)#%d (1) {
   ["static"]=>
@@ -16,4 +15,3 @@ object(Closure)#%d (1) {
     *RECURSION*
   }
 }
-===DONE===

Zend/tests/closures/closure_from_callable_basic.phpt

@@ -96,7 +96,6 @@ echo $fn(" OK".PHP_EOL);
 
 
 ?>
-===DONE===
 --EXPECT--
 Access public static function OK
 Access public static function with different case OK
@@ -118,4 +117,3 @@ Access public instance method of parent object through "self::" to parent method
 Access proteced instance method of parent object through "self::" to parent method OK
 MagicCall __call instance method __call,nonExistentMethod, OK
 MagicCall __callStatic static method __callStatic,nonExistentMethod, OK
-===DONE===