Page MenuHomePhabricator

Separate map building and updating in `PhutilLibraryMapBuilder`
ClosedPublic

Authored by joshuaspence on Jul 9 2014, 9:10 PM.
Tags
None
Referenced Files
F15466053: D9861.diff
Thu, Apr 3, 1:06 AM
F15434520: D9861.id.diff
Tue, Mar 25, 4:14 AM
F15424091: D9861.id23666.diff
Sat, Mar 22, 8:07 PM
F15422452: D9861.diff
Sat, Mar 22, 8:29 AM
F15404089: D9861.id.diff
Tue, Mar 18, 6:10 AM
F15402873: D9861.diff
Tue, Mar 18, 12:47 AM
F15284404: D9861.diff
Mar 4 2025, 9:41 AM
Unknown Object (File)
Feb 25 2025, 5:08 PM
Subscribers

Details

Summary

Separate the building of the library map from the writing of the library map to file. This is a much nicer approach.

Test Plan

Ran ./scripts/phutil_rebuild_map.php src and ./scripts/phutil_rebuild_map.php --show src.

Diff Detail

Repository
rPHU libphutil
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

joshuaspence retitled this revision from to Separate map building and updating in `PhutilLibraryMapBuilder`.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
joshuaspence edited edge metadata.

Also update PhutilInfrastructureTestCase

epriestley edited edge metadata.

Maybe call updateMap() something like writeMap() or updateAndWriteMap(), it's not as clear as it could be from the outside that "update" implies "write to disk".

This revision is now accepted and ready to land.Jul 9 2014, 9:31 PM

Well, writeMap would be confusing with writeLibraryMap. Maybe buildMap() to generate + return, and buildAndWriteMap() to generate + write.

joshuaspence edited edge metadata.
  • Rename updateMap to updateAndWriteMap