Paths

Table of Contentst

Minor fix for JSONLint
ClosedPublic
Actions

Authored by joshuaspence on Sep 9 2014, 1:40 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, May 19, 8:27 AM

	Unknown Object (File)
	Apr 27 2025, 8:57 AM

	Unknown Object (File)
	Apr 12 2025, 1:36 AM

	Unknown Object (File)
	Apr 10 2025, 12:25 PM

	Unknown Object (File)
	Apr 10 2025, 12:25 PM

	Unknown Object (File)
	Mar 31 2025, 12:40 PM

	Unknown Object (File)
	Mar 20 2025, 1:56 AM

	Unknown Object (File)
	Mar 19 2025, 11:18 PM

View All Files

Subscribers

epriestley

Korvin

Details

Reviewers

epriestley

Group Reviewers

Blessed Reviewers

Commits

rPHU1a59ecfe122a: Minor fix for JSONLint

Summary

In D10435, the JSONLint external was patched in an attempt to ensure PHP 5.2 compatibility. The fact that $this->lexer->lex() will produce a different token if invoked a second time was overlooked.

Test Plan

Ran ./bin/arc lint -- src/lint/linter in rARC.

Diff Detail

Repository

rPHU libphutil

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

joshuaspence updated this revision to Diff 25131.Sep 9 2014, 1:40 PM

joshuaspence retitled this revision from to Minor fix for JSONLint.

joshuaspence updated this object.

joshuaspence edited the test plan for this revision. (Show Details)

joshuaspence added a reviewer: epriestley.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptSep 9 2014, 1:40 PM

Herald added subscribers: Korvin, epriestley. · View Herald Transcript

Harbormaster completed remote builds in B2472: Diff 25131.Sep 9 2014, 1:41 PM

epriestley accepted this revision.Sep 9 2014, 2:02 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Sep 9 2014, 2:02 PM

Closed by commit rPHU1a59ecfe122a (authored by @joshuaspence).

Revision Contents
Changeset List

Path

Size

externals/

jsonlint/

src/

Seld/

JsonLint/

JsonParser.php

5 lines

Diff 25132

View Options

externals/jsonlint/src/Seld/JsonLint/JsonParser.php

Minor fix for JSONLintClosedPublicActions