Page MenuHomePhabricator

Fix "undefined index" error in Differential changeset parser
AbandonedPublic

Authored by epriestley on Jul 31 2017, 7:56 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 11, 10:53 AM
Unknown Object (File)
Tue, Apr 2, 8:56 AM
Unknown Object (File)
Sun, Mar 31, 7:29 AM
Unknown Object (File)
Feb 22 2024, 1:21 PM
Unknown Object (File)
Feb 22 2024, 10:15 AM
Unknown Object (File)
Feb 3 2024, 9:17 PM
Unknown Object (File)
Feb 3 2024, 4:24 PM
Unknown Object (File)
Jan 18 2024, 7:19 AM
Subscribers

Details

Summary

Fixes T8591. It seems that this error occurs when a changeset has a synthetic comment linked to a line number that doesn't exist.

Test Plan

Added a test case, which failed with an "undefined index" error before this patch.

Diff Detail

Repository
rP Phabricator
Branch
changeset
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 17847
Build 23970: Run Core Tests
Build 23969: arc lint + arc unit