Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(432)

Unified Diff: lib/src/codegen/code_generator.dart

Issue 1788973002: Remove code that requires whole-program compile (Closed) Base URL: git@github.com:dart-lang/dev_compiler.git@master
Patch Set: merged Created 4 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « lib/runtime/messages_widget.js.map ('k') | lib/src/codegen/html_codegen.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: lib/src/codegen/code_generator.dart
diff --git a/lib/src/codegen/code_generator.dart b/lib/src/codegen/code_generator.dart
deleted file mode 100644
index c7a3c973413d7b543e0fa4f8941e32dac661fe3f..0000000000000000000000000000000000000000
--- a/lib/src/codegen/code_generator.dart
+++ /dev/null
@@ -1,124 +0,0 @@
-// Copyright (c) 2015, the Dart project authors. Please see the AUTHORS file
-// for details. All rights reserved. Use of this source code is governed by a
-// BSD-style license that can be found in the LICENSE file.
-
-import 'package:analyzer/dart/element/element.dart'
- show CompilationUnitElement, LibraryElement;
-import 'package:analyzer/src/generated/engine.dart' show AnalysisContext;
-import 'package:path/path.dart' as path;
-
-import '../compiler.dart' show AbstractCompiler;
-import '../info.dart';
-import '../utils.dart' show canonicalLibraryName;
-import '../options.dart' show CodegenOptions;
-
-abstract class CodeGenerator {
- final AbstractCompiler compiler;
- final AnalysisContext context;
- final CodegenOptions options;
-
- CodeGenerator(AbstractCompiler compiler)
- : compiler = compiler,
- context = compiler.context,
- options = compiler.options.codegenOptions;
-
- /// Return a hash, if any, that can be used for caching purposes. When two
- /// invocations to this function return the same hash, the underlying
- /// code-generator generated the same code.
- String generateLibrary(LibraryUnit unit);
-
- static List<String> _searchPaths = () {
- // TODO(vsm): Can we remove redundancy with multi_package_resolver logic?
- var packagePaths =
- new String.fromEnvironment('package_paths', defaultValue: null);
- if (packagePaths == null) return null;
- var list = packagePaths.split(',');
- // Normalize the paths.
- list = new List<String>.from(list.map(_dirToPrefix));
- // The current directory is implicitly in the search path.
- list.add(_dirToPrefix(path.current));
- // Sort by reverse length to prefer longer prefixes.
- // This ensures that we get the minimum valid suffix. E.g., if we have:
- // - root/
- // - root/generated/
- // in our search path, and the path "root/generated/foo/bar.dart", we'll
- // compute "foo/bar.dart" instead of "generated/foo/bar.dart" in the search
- // below.
- list.sort((s1, s2) => s2.length - s1.length);
- return list;
- }() as List<String>;
-
- static String _dirToPrefix(String dir) {
- dir = path.absolute(dir);
- dir = path.normalize(dir);
- if (!dir.endsWith(path.separator)) {
- dir = dir + path.separator;
- }
- return dir;
- }
-
- static String _convertIfPackage(String dir) {
- assert(path.isRelative(dir));
- var parts = path.split(dir);
- var index = parts.indexOf('lib');
- if (index < 0) {
- // Not a package.
- return dir;
- }
- // This is a package.
- // Convert: foo/bar/lib/baz/hi
- // to: packages/foo.bar/baz/hi
- var packageName = parts.sublist(0, index).join('.');
- var prefix = path.join('packages', packageName);
- var suffix = path.joinAll(parts.sublist(index + 1));
- return path.join(prefix, suffix);
- }
-
- static String _getOutputDirectory(
- String name, CompilationUnitElement unitElement) {
- var uri = unitElement.source.uri.toString();
- String suffix;
- if (uri.startsWith('dart:') || _searchPaths == null) {
- // Use the library name as the directory.
- suffix = name;
- } else if (uri.startsWith('package:')) {
- suffix = uri.replaceFirst('package:', 'packages/');
- suffix = path.dirname(suffix);
- } else {
- // Recover the original search path and use the relative path
- // from there as the directory.
- // TODO(vsm): Is there a better way to get the resolved path?
- var resolvedPath = unitElement.toString();
- var resolvedDir = path.dirname(resolvedPath);
- for (var prefix in _searchPaths) {
- if (resolvedDir.startsWith(prefix)) {
- suffix = resolvedDir.substring(prefix.length);
- break;
- }
- }
- }
- assert(suffix != null);
- assert(path.isRelative(suffix));
- return _convertIfPackage(suffix);
- }
-
- static Uri uriFor(LibraryElement lib) {
- var unitElement = lib.definingCompilationUnit;
- var uri = unitElement.source.uri;
- if (uri.scheme == 'dart') return uri;
- if (uri.scheme == 'package') return uri;
- var suffix = _getOutputDirectory(canonicalLibraryName(lib), unitElement);
- suffix = path.join(suffix, uri.pathSegments.last);
- var parts = path.split(suffix);
- var index = parts.indexOf('packages');
- if (index < 0) {
- // Not a package.
- // TODO(leafp) These may need to be adjusted
- // relative to the import location
- return new Uri(path: suffix);
- }
- assert(index == 0);
- return new Uri(
- scheme: 'package', path: path.joinAll(parts.sublist(index + 1)));
- }
-}
« no previous file with comments | « lib/runtime/messages_widget.js.map ('k') | lib/src/codegen/html_codegen.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698