Homestyx hydra
Diffusion hydra b3d1ecebc7f8

Fix another issue with line order on unified diffs
b3d1ecebc7f8
Actions

Description

Fix another issue with line order on unified diffs

Summary: This improves some cases with interleaved added and removed lines, and adds test coverage.

Test Plan:

  • Added and executed unit tests.
  • Viewed raw diff and saw sensible/expected output.

Reviewers: btrahan

Reviewed By: btrahan

Subscribers: epriestley

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

Details

Provenance
epriestleyAuthored on Mar 5 2015, 7:24 PM
sirocylPushed on Oct 16 2024, 5:49 AM
Parents
R1:c97040de4707: Conpherence - get the durable column header actions working correctly
Branches
Unknown
Tags
Unknown

Event Timeline

Changes (5)

PathSize
src/
applications/
differential/
__tests__/
data/
render/

R1:b3d1ecebc7f8

View Options

src/applications/differential/__tests__/data/order.diff

Loading...
View Options

src/applications/differential/__tests__/data/order.diff.one.expect

Loading...
View Options

src/applications/differential/__tests__/data/order.diff.two.expect

Loading...
View Options

src/applications/differential/render/DifferentialChangesetRenderer.php

Loading...
View Options

src/applications/differential/render/DifferentialChangesetTestRenderer.php

Loading...
styx copyright 2020 sirocyl / ellie (kymkdd) / ncommander / styx contributors
Use of other names and brands is not intended to reflect a claim of, or right to, use the name or brand.
styx is an operating system using the Linux kernel and components, and "styx Linux" as a phrase does not reflect the inclusion of the Linux brand or trademark in the styx project.
Linux® is the registered trademark of Linus Torvalds in the U.S. and other countries.
Terms and Conditions (TODO: fix this!)