Page MenuHomePhabricator
Files F13220935

D9953.diff
No OneTemporary
Actions

D9953.diff
View Options

diff --git a/src/moduleutils/PhutilLibraryMapBuilder.php b/src/moduleutils/PhutilLibraryMapBuilder.php
--- a/src/moduleutils/PhutilLibraryMapBuilder.php
+++ b/src/moduleutils/PhutilLibraryMapBuilder.php
@@ -15,6 +15,9 @@
private $quiet = true;
private $subprocessLimit = 8;
+ private $fileSymbolMap;
+ private $librarySymbolMap;
+
const LIBRARY_MAP_VERSION_KEY = '__library_version__';
const LIBRARY_MAP_VERSION = 2;
@@ -62,78 +65,44 @@
}
/**
- * Build the library map.
+ * Get the map of symbols in this library, analyzing the library to build it
+ * if necessary.
*
- * @return dict
+ * @return map<string, wild> Information about symbols in this library.
*
* @task map
*/
public function buildMap() {
- // Identify all the ".php" source files in the library.
- $this->log("Finding source files...\n");
- $source_map = $this->loadSourceFileMap();
- $this->log("Found ".number_format(count($source_map))." files.\n");
-
- // Load the symbol cache with existing parsed symbols. This allows us
- // to remap libraries quickly by analyzing only changed files.
- $this->log("Loading symbol cache...\n");
- $symbol_cache = $this->loadSymbolCache();
-
- // Build out the symbol analysis for all the files in the library. For
- // each file, check if it's in cache. If we miss in the cache, do a fresh
- // analysis.
- $symbol_map = array();
- $futures = array();
- foreach ($source_map as $file => $hash) {
- if (!empty($symbol_cache[$hash])) {
- $symbol_map[$file] = $symbol_cache[$hash];
- continue;
- }
- $futures[$file] = $this->buildSymbolAnalysisFuture($file);
+ if ($this->librarySymbolMap === null) {
+ $this->analyzeLibrary();
}
- $this->log("Found ".number_format(count($symbol_map))." files in cache.\n");
-
- // Run the analyzer on any files which need analysis.
- if ($futures) {
- $limit = $this->subprocessLimit;
- $count = number_format(count($futures));
+ return $this->librarySymbolMap;
+ }
- $this->log("Analyzing {$count} files with {$limit} subprocesses...\n");
- $progress = new PhutilConsoleProgressBar();
- if ($this->quiet) {
- $progress->setQuiet(true);
- }
- $progress->setTotal(count($futures));
-
- foreach (Futures($futures)->limit($limit) as $file => $future) {
- $result = $future->resolveJSON();
- if (empty($result['error'])) {
- $symbol_map[$file] = $result;
- } else {
- $progress->done(false);
- throw new XHPASTSyntaxErrorException(
- $result['line'],
- $file.': '.$result['error']);
- }
- $progress->update(1);
- }
- $progress->done();
+ /**
+ * Get the map of files in this library, analyzing the library to build it
+ * if necessary.
+ *
+ * Returns a map of file paths to information about symbols used and defined
+ * in the file.
+ *
+ * @return map<string, wild> Information about files in this library.
+ *
+ * @task map
+ */
+ public function buildFileSymbolMap() {
+ if ($this->fileSymbolMap === null) {
+ $this->analyzeLibrary();
}
-
- // We're done building the cache, so write it out immediately. Note that
- // we've only retained entries for files we found, so this implicitly cleans
- // out old cache entries.
- $this->writeSymbolCache($symbol_map, $source_map);
-
- // Our map is up to date, so either show it on stdout or write it to disk.
- $this->log("Building library map...\n");
- return $this->buildLibraryMap($symbol_map);
+ return $this->fileSymbolMap;
}
/**
* Build and update the library map.
*
+ * @return void
+ *
* @task map
*/
public function buildAndWriteMap() {
@@ -441,4 +410,77 @@
Filesystem::writeFile($map_file, $source_file);
}
+
+ /**
+ * Analyze the library, generating the file and symbol maps.
+ *
+ * @return void
+ */
+ private function analyzeLibrary() {
+ // Identify all the ".php" source files in the library.
+ $this->log("Finding source files...\n");
+ $source_map = $this->loadSourceFileMap();
+ $this->log("Found ".number_format(count($source_map))." files.\n");
+
+ // Load the symbol cache with existing parsed symbols. This allows us
+ // to remap libraries quickly by analyzing only changed files.
+ $this->log("Loading symbol cache...\n");
+ $symbol_cache = $this->loadSymbolCache();
+
+ // Build out the symbol analysis for all the files in the library. For
+ // each file, check if it's in cache. If we miss in the cache, do a fresh
+ // analysis.
+ $symbol_map = array();
+ $futures = array();
+ foreach ($source_map as $file => $hash) {
+ if (!empty($symbol_cache[$hash])) {
+ $symbol_map[$file] = $symbol_cache[$hash];
+ continue;
+ }
+ $futures[$file] = $this->buildSymbolAnalysisFuture($file);
+ }
+ $this->log("Found ".number_format(count($symbol_map))." files in cache.\n");
+
+ // Run the analyzer on any files which need analysis.
+ if ($futures) {
+ $limit = $this->subprocessLimit;
+ $count = number_format(count($futures));
+
+ $this->log("Analyzing {$count} files with {$limit} subprocesses...\n");
+
+ $progress = new PhutilConsoleProgressBar();
+ if ($this->quiet) {
+ $progress->setQuiet(true);
+ }
+ $progress->setTotal(count($futures));
+
+ foreach (Futures($futures)->limit($limit) as $file => $future) {
+ $result = $future->resolveJSON();
+ if (empty($result['error'])) {
+ $symbol_map[$file] = $result;
+ } else {
+ $progress->done(false);
+ throw new XHPASTSyntaxErrorException(
+ $result['line'],
+ $file.': '.$result['error']);
+ }
+ $progress->update(1);
+ }
+ $progress->done();
+ }
+
+ $this->fileSymbolMap = $symbol_map;
+
+ // We're done building the cache, so write it out immediately. Note that
+ // we've only retained entries for files we found, so this implicitly cleans
+ // out old cache entries.
+ $this->writeSymbolCache($symbol_map, $source_map);
+
+ // Our map is up to date, so either show it on stdout or write it to disk.
+ $this->log("Building library map...\n");
+
+ $this->librarySymbolMap = $this->buildLibraryMap($symbol_map);
+ }
+
+
}

File Metadata

Mime Type
text/plain
Expires
Mon, May 20, 2:07 AM (4 w, 14 h ago)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
6289214
Default Alt Text
D9953.diff (6 KB)

Event Timeline

Phabricator · Built by Phacility