Page MenuHomePhabricator
Differential D9287

Move more hunk loads into DifferentialHunkQuery
ClosedPublic
Actions

Authored by epriestley on May 25 2014, 3:11 PM.
Tags
None
Referenced Files
F15449375: D9287.id.diff
Fri, Mar 28, 9:49 AM
F15448460: D9287.id22052.diff
Fri, Mar 28, 5:02 AM
F15444723: D9287.diff
Thu, Mar 27, 10:53 AM
F15430550: D9287.id.diff
Mon, Mar 24, 8:06 AM
F15395037: D9287.diff
Sun, Mar 16, 2:55 AM
F15381089: D9287.diff
Fri, Mar 14, 5:48 AM
F15284538: D9287.id22326.diff
Tue, Mar 4, 9:57 AM
Unknown Object (File)
Sat, Mar 1, 1:40 PM
View All Files
Subscribers
epriestley

Details

Reviewers
btrahan
Maniphest Tasks
T4045: Store diffs as binary, not UTF-8
T5179: Unearth Differential rendering technical debt
Commits
Restricted Diffusion Commit
rP71e9fb96b56e: Move more hunk loads into DifferentialHunkQuery
Summary

Ref T5179. Ref T4045. I want to move all hunk loads into DifferentialHunkQuery so I can make it do magical things where hunks come from multiple places, handle non-utf8 encodings properly, handle compression, archive into Files, and so on.

Test Plan

Viewed some revisions. Called differential.getrawdiff.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley updated this revision to Diff 22052.May 25 2014, 3:11 PM
epriestley retitled this revision from to Move more hunk loads into DifferentialHunkQuery.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
epriestley added tasks: T5179: Unearth Differential rendering technical debt, T4045: Store diffs as binary, not UTF-8.
Herald added a subscriber: epriestley. · View Herald TranscriptMay 25 2014, 3:11 PM
Harbormaster completed remote builds in B695: Diff 22052.May 25 2014, 3:13 PM
btrahan accepted this revision.Jun 3 2014, 10:10 PM
btrahan edited edge metadata.
This revision is now accepted and ready to land.Jun 3 2014, 10:10 PM
epriestley closed this revision.Jun 4 2014, 1:03 AM
epriestley updated this revision to Diff 22326.

Closed by commit rP71e9fb96b56e (authored by @epriestley).

Revision Contents
Changeset List

PathSize
src/
6 lines
applications/
differential/
conduit/
16 lines
parser/
__tests__/
21 lines
query/
33 lines
render/
4 lines
storage/
28 lines

Diff 22326

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/differential/conduit/ConduitAPI_differential_getrawdiff_Method.php

Loading...
View Options

src/applications/differential/parser/__tests__/DifferentialHunkParserTestCase.php

Loading...
View Options

src/applications/differential/query/DifferentialDiffQuery.php

Loading...
View Options

src/applications/differential/render/DifferentialRawDiffRenderer.php

Loading...
View Options

src/applications/differential/storage/DifferentialChangeset.php

Loading...
Phabricator · Built by Phacility