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
F15382665: D18302.diff
Fri, Mar 14, 1:43 PM
Unknown Object (File)
Thu, Feb 20, 10:21 PM
Unknown Object (File)
Feb 5 2025, 7:26 PM
Unknown Object (File)
Jan 27 2025, 5:39 AM
Unknown Object (File)
Jan 21 2025, 11:53 AM
Unknown Object (File)
Dec 24 2024, 3:24 AM
Unknown Object (File)
Dec 16 2024, 7:13 PM
Unknown Object (File)
Dec 5 2024, 2:40 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