Fix handling of named label for goto.

[TysonAndre]

The label is a standalone statement just like any other statement.
It isn't associated with the next statement by PHP's parser.

E.g. while (false) label: echo "test\n"; echoes "test".

Fixes #154

[TysonAndre]

Actually, I think it's incorrect for a goto label to be associated with a statement.

php > while(false) label: echo "test";
test

That example gets parsed equivalently to while(false) { label: } echo "test"; by php, but tolerant-php-parser incorrectly treats it as while (false) {label: echo "test"}

Looking at the php-src parser implementation, a goto label is a standalone statement just like any other statement

	|	';'	/* empty statement */ { $$ = NULL; }
	|	T_TRY '{' inner_statement_list '}' catch_list finally_statement
			{ $$ = zend_ast_create(ZEND_AST_TRY, $3, $5, $6); }
	|	T_GOTO T_STRING ';' { $$ = zend_ast_create(ZEND_AST_GOTO, $2); }
	|	T_STRING ':' { $$ = zend_ast_create(ZEND_AST_LABEL, $1); }
;

EDIT: The php-src implementation is also consistent with the documented https://github.com/php/php-langspec/blob/master/spec/11-statements.md#grammar-named-label-statement specification

php-language-server doesn't seem to be affected by this change because NamedLabelStatement is handled like any other node - it should fix the bugs mentioned in the linked ticket
.
Phan hardcodes an exact version (e.g. "0.0.22", so it is also unaffected because it'll be fixed when the release is manually updated)

[roblourens]

Good catch! I wouldn't have expected that behavior.