Skip to content

Commit 37b43b3

Browse files
committed
main: simplify calling makePromise()
1 parent 5dc6ad1 commit 37b43b3

File tree

2 files changed

+13
-9
lines changed

2 files changed

+13
-9
lines changed

main/promise.c

Lines changed: 11 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -71,15 +71,21 @@ int makePromise (const char *parser,
7171
int r;
7272
langType lang = LANG_IGNORE;
7373

74+
const bool is_thin_stream_spec =
75+
isThinStreamSpec(startLine, startCharOffset,
76+
endLine, endCharOffset,
77+
sourceLineOffset);
78+
79+
if (!is_thin_stream_spec
80+
&& (startLine > endLine
81+
|| (startLine == endLine && startCharOffset >= endCharOffset)))
82+
return -1;
83+
7484
verbose("makePromise: %s start(line: %lu, offset: %ld, srcline: %lu), end(line: %lu, offset: %ld)\n",
7585
parser? parser: "*", startLine, startCharOffset, sourceLineOffset,
7686
endLine, endCharOffset);
7787

78-
if ((!isThinStreamSpec(startLine,
79-
startCharOffset,
80-
endLine,
81-
endCharOffset,
82-
sourceLineOffset))
88+
if ((!is_thin_stream_spec)
8389
&& ( !isXtagEnabled (XTAG_GUEST)))
8490
return -1;
8591

parsers/php.c

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -913,10 +913,8 @@ static void readToken (tokenInfo *const token)
913913
unsigned long endLineNumber = getInputLineNumber ();
914914
int endLineOffset = getInputLineOffset ();
915915

916-
if ((startLineNumber != endLineNumber)
917-
|| (startLineOffset != endLineOffset))
918-
makePromise ("HTML", startLineNumber, startLineOffset,
919-
endLineNumber, endLineOffset, startSourceLineNumber);
916+
makePromise ("HTML", startLineNumber, startLineOffset,
917+
endLineNumber, endLineOffset, startSourceLineNumber);
920918
}
921919
else
922920
c = getcFromInputFile ();

0 commit comments

Comments
 (0)