HomePhabricator

Fix crazystairs with non-stairlike reverse-indented lists

Description

Fix crazystairs with non-stairlike reverse-indented lists

Summary: Ref T5102. This loop is supposed to select the first item at the minimum depth, but uses < rather than <= so it selects the last item instead.

Test Plan:
Added and ran unit tests. Viewed stairs in preview, they worked correctly.

Screen_Shot_2014-07-24_at_4.17.29_PM.png (543×418 px, 24 KB)

Reviewers: asherkin, btrahan, chad

Reviewed By: chad

Subscribers: epriestley

Maniphest Tasks: T5102

Differential Revision: https://secure.phabricator.com/D10043

Details

Provenance
epriestleyAuthored on
epriestleyPushed on Jul 25 2014, 4:58 AM
Reviewer
chad
Differential Revision
D10043: Fix crazystairs with non-stairlike reverse-indented lists
Parents
rPHU4298b0da1573: Choose a better class name for PhutilMethodNotImplementedException
Branches
Unknown
Tags
Unknown

Event Timeline