Paths

Table of Contentst

Differential D20605

Fix two "msort()" vs "msortv()" issues in "arc land"
ClosedPublic
Actions

Authored by epriestley on Jun 20 2019, 9:41 PM.

Tags

None

Referenced Files

	F22717094: D20605.id49150.diff
	Sat, Oct 3, 6:50 PM

	F22635980: D20605.id49141.diff
	Wed, Sep 30, 7:14 AM

	F22635976: D20605.id49141.diff
	Wed, Sep 30, 7:14 AM

	F22530381: D20605.diff
	Mon, Sep 21, 2:51 PM

	Unknown Object (File)
	Tue, Sep 15, 6:40 PM

	Unknown Object (File)
	Aug 10 2026, 6:05 AM

	Unknown Object (File)
	Jul 4 2026, 3:04 PM

	Unknown Object (File)
	Jun 30 2026, 4:52 AM

View All Files

Subscribers

None

Details

Reviewers

amckinley

Maniphest Tasks

T13303: There is an extra or missing "array_reverse()" somewhere after recent transaction strength sorting changes

Commits

rARCd92fa96366c0: Fix two "msort()" vs "msortv()" issues in "arc land"

Summary

See https://github.com/phacility/arcanist/pull/242. Ref T13303. A little more fallout turned up in arc.

Test Plan

grep msort(

Diff Detail

Repository

rARC Arcanist

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Jun 20 2019, 9:41 PM

Harbormaster completed remote builds in B23013: Diff 49141.Jun 20 2019, 9:41 PM

epriestley requested review of this revision.Jun 20 2019, 9:41 PM

amckinley accepted this revision.Jun 20 2019, 10:10 PM

This revision is now accepted and ready to land.Jun 20 2019, 10:10 PM

Closed by commit rARCd92fa96366c0: Fix two "msort()" vs "msortv()" issues in "arc land" (authored by epriestley). · Explain WhyJun 20 2019, 11:07 PM

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rARCd92fa96366c0: Fix two "msort()" vs "msortv()" issues in "arc land".

Revision Contents
Changeset List

Path

Size

src/

workflow/

ArcanistLandWorkflow.php

4 lines

Diff 49150

View Options

src/workflow/ArcanistLandWorkflow.php

Fix two "msort()" vs "msortv()" issues in "arc land"ClosedPublicActions