Don't ltrim when preserving formatting

We shouldn't ltrim when printing a whole file, that way we will
not just fail to preserve formatting, but actually change semantics
by dropping

Don't ltrim when preserving formatting

We shouldn't ltrim when printing a whole file, that way we will
not just fail to preserve formatting, but actually change semantics
by dropping meaningful whitespace.

show more ...

Remove garbage collection section from docs

The GC issue has been fixed in PHP 7.3 and is no longer relevant.

Don't generate braces for "else if"

Mentioned in #915.

Release PHP-Parser 5.0.0 alpha 2

Update upgrading documentation

Update changelog

Add PrettyPrinter interface

Closes #423.

Revert "Rename PrettyPrinterAbstract to PrettyPrinter"

This reverts commit 2217f14d6e039f1c0572329e6fcc99f6c17178a3.

Set allowed exceptions in fuzzer

Don't always omit parentheses for argument-less yield

We may need parentheses around an argument-less yield to distinguish
(yield) - $a from yield -$a and similar.

Treat argumen

Don't always omit parentheses for argument-less yield

We may need parentheses around an argument-less yield to distinguish
(yield) - $a from yield -$a and similar.

Treat argument-less yield like a prefix operator. This will print
parentheses a bit more often than is really required, because the
arity ambiguity only exists for certain operators. This seems like
a reasonably good approximation through, as it only affects
argument-less yield on the LHS of infix operators.

show more ...

Handle group use special case in fuzzer

Same as for normal use.

Respect namespace style for enum scalar type printing

Doesn't matter in any practical sense because such types are always
invalid, but makes sure pretty printing round-trips.

Handle overflowing \u escape sequence

Don't skip tests with evaluate segment when generating corpus

This code was adopted from test updating, but in this context
we're fine with just evaluating these.

Produce error if <?= used as identifier

Due to a peculiarity of how <?= is handled, it can get treated as
an echo identifier. Make sure it isn't, to match PHP behavior.

Rename PrettyPrinterAbstract to PrettyPrinter

Strip trailing doc string newline before parsing escape sequences

If the doc string ends on an escaped \r, it should not get eaten
as the "last newline".

Support parsing from stdin in php-parse

Following the usual convention, read from stdin if the file name
is "-".

Detect another special case in fuzzer

Handle isolated \r in doc string

Doc strings have a trailing \n and these will get interpreted as
\r\n and removed from the string contents.

For nowdoc, fall back to single quot

Handle isolated \r in doc string

Doc strings have a trailing \n and these will get interpreted as
\r\n and removed from the string contents.

For nowdoc, fall back to single quote if there's a trailing \r.
For heredoc, escape all isolated \r -- unlike \n and \r\n this is
really a special character, because this is no longer relevant as
an actual newline character.

show more ...

Handle another yield edge case

match also uses =>, so we need to make sure that a trailing yield
is wrapped in parentheses.

Prevent merging of consecutive -/+ more thoroughly

The unary -/+ or --/++ might not be the direct child. Instead
determine this by actually printing the operand and checking
whether

Prevent merging of consecutive -/+ more thoroughly

The unary -/+ or --/++ might not be the direct child. Instead
determine this by actually printing the operand and checking
whether it starts with -/+.

show more ...

Fix logic for new operand parentheses requirement

We need to perform this check recursively.

Add fuzzing target

Support yield precedence

Since PHP 7.0 yield is a proper expression, so print it with
proper precedence. If the pretty printer is configured for
older version (non-default), then alw

Support yield precedence

Since PHP 7.0 yield is a proper expression, so print it with
proper precedence. If the pretty printer is configured for
older version (non-default), then always print parentheses.

There is an interesting interaction here, in that => is resolved
in favor of yield if the yield occurs as part of an array (or
outer yield). This kind of bypasses the entire precedence hierarchy
and *only* affects yield expressions. For the sake of simplicity
this is modeled via normal LHS precedence, because this will only
add unnecessary parentheses to a handful of low precedence unary
operators. If desired, a special marker for this purpose could be
added though.

show more ...