Page MenuHomePhabricator

Fix an unusual nonterminating task graph node
ClosedPublic

Authored by epriestley on Jan 17 2017, 4:40 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 25, 2:12 AM
Unknown Object (File)
Fri, Apr 19, 6:07 PM
Unknown Object (File)
Mon, Apr 8, 1:54 AM
Unknown Object (File)
Sat, Mar 30, 3:30 AM
Unknown Object (File)
Fri, Mar 29, 1:54 PM
Unknown Object (File)
Mar 21 2024, 8:04 AM
Unknown Object (File)
Mar 20 2024, 6:21 AM
Unknown Object (File)
Mar 19 2024, 10:24 PM
Subscribers
None

Details

Summary

Fixes T12114. There were a couple of bugs here:

  • We could draw too many joining lines if a node had a parent with multiple descendants.
  • We could incorrectly ignore columns because of an unset().

I think this fixes both things without collateral damage. This whole thing is a little hard to understand/debug and has grown beyond its original scope, so I'll probably rewrite it if there are more issues.

Test Plan
  • Unit tests.
  • My local repro is clean now:

Screen Shot 2017-01-17 at 8.36.55 AM.png (349×262 px, 30 KB)

Diff Detail

Repository
rP Phabricator
Branch
graph1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 15286
Build 20114: Run Core Tests
Build 20113: arc lint + arc unit

Event Timeline

This revision is now accepted and ready to land.Jan 17 2017, 4:42 PM
This revision was automatically updated to reflect the committed changes.