Don’t use print_r

sabberworm · sabberworm · commit ee5073f6a04b · 2013-04-17T20:24:07.000+02:00
It sometimes conflicts with output buffering
diff --git a/lib/Sabberworm/CSS/Parser.php b/lib/Sabberworm/CSS/Parser.php
@@ -542,7 +542,7 @@ private function consumeUntil($aEnd, $bIncludeEnd = false) {
 			}
 		}
 		if ($iEndPos === null) {
-			throw new UnexpectedTokenException(print_r($aEnd, true), $this->peek(5), 'search');
+			throw new UnexpectedTokenException('One of ("'.implode('","', $aEnd).'")', $this->peek(5), 'search');
 		}
 		return $this->consume($iEndPos - $this->iCurrentPosition);
 	}

Original file line number	Diff line number	Diff line change
`@@ -542,7 +542,7 @@ private function consumeUntil($aEnd, $bIncludeEnd = false) {`
`542`	`542`	`}`
`543`	`543`	`}`
`544`	`544`	`if ($iEndPos === null) {`
`545`		`- throw new UnexpectedTokenException(print_r($aEnd, true), $this->peek(5), 'search');`
	`545`	`+ throw new UnexpectedTokenException('One of ("'.implode('","', $aEnd).'")', $this->peek(5), 'search');`
`546`	`546`	`}`
`547`	`547`	`return $this->consume($iEndPos - $this->iCurrentPosition);`
`548`	`548`	`}`