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

Side by Side Diff: tools/html_json_doc/lib/json_to_html.dart

Issue 11413197: Updated json_to_html to be sync. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 8 years 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 /** 5 /**
6 * Library for taking a JSON file and putting the comments located within into 6 * Library for taking a JSON file and putting the comments located within into
7 * the HTML files the comments are associated with. 7 * the HTML files the comments are associated with.
8 * 8 *
9 * The format of the JSON file is: 9 * The format of the JSON file is:
10 * 10 *
(...skipping 48 matching lines...) Expand 10 before | Expand all | Expand 10 after
59 _anyErrors = true; 59 _anyErrors = true;
60 completer.complete(false); 60 completer.complete(false);
61 } else { 61 } else {
62 fileJson = JSON.parse(jsonRead); 62 fileJson = JSON.parse(jsonRead);
63 } 63 }
64 64
65 // TODO(amouravski): Refactor to not duplicate code here and in html-to-json. 65 // TODO(amouravski): Refactor to not duplicate code here and in html-to-json.
66 // Find html files. (lister) 66 // Find html files. (lister)
67 final lister = htmlDir.list(recursive: false); 67 final lister = htmlDir.list(recursive: false);
68 68
69 var pathList = <Path>[];
69 lister.onFile = (String path) { 70 lister.onFile = (String path) {
71 pathList.add(path);
72 };
73
74 getFile(Path path) {
70 final name = new Path.fromNative(path).filename; 75 final name = new Path.fromNative(path).filename;
71 76
72 // Ignore private classes. 77 // Ignore private classes.
73 if (name.startsWith('_')) return; 78 if (name.startsWith('_')) return;
74 79
75 // Ignore non-dart files. 80 // Ignore non-dart files.
76 if (!name.endsWith('.dart')) return; 81 if (!name.endsWith('.dart')) return;
77 82
78 File file = new File(path); 83 File file = new File(path);
79 84
80 // TODO(amouravski): Handle missing file. 85 // TODO(amouravski): Handle missing file.
81 if (!file.existsSync()) { 86 if (!file.existsSync()) {
82 print('ERROR: cannot find file: $path'); 87 print('ERROR: cannot find file: $path');
83 _anyErrors = true; 88 _anyErrors = true;
84 return; 89 return;
85 } 90 }
86 91
87 if (!fileJson.containsKey(name)) { 92 if (!fileJson.containsKey(name)) {
88 print('WARNING: file found that is not in JSON: $path'); 93 print('WARNING: file found that is not in JSON: $path');
89 _anyErrors = true; 94 _anyErrors = true;
90 return; 95 return;
91 } 96 }
92 97
93 var comments = fileJson[name]; 98 var comments = fileJson[name];
94 99
95 _convertFile(file, comments); 100 print('BLAH $name');
96 101
97 fileJson.remove(name); 102 fileJson.remove(name);
98 }; 103 };
99 104
100 lister.onDone = (_) { 105 lister.onDone = (_) {
106 while(!pathList.isEmpty) {
107 getFile(pathList.removeLast());
108 }
109 print('foo');
101 110
102 fileJson.forEach((key, _) { 111 fileJson.forEach((key, _) {
103 print('WARNING: the following filename was found in the JSON but not in ' 112 print('WARNING: the following filename was found in the JSON but not in '
104 '${htmlDir.path}:\n"$key"'); 113 '${htmlDir.path}:\n"$key"');
105 _anyErrors = true; 114 _anyErrors = true;
106 }); 115 });
107 116
108 completer.complete(_anyErrors); 117 completer.complete(_anyErrors);
109 };
Adam 2012/11/27 22:18:05 This closing brace is needed
Andrei Mouravski 2012/11/28 00:28:34 Done.
110 118
111 return completer.future; 119 return completer.future;
112 } 120 }
113 121
114 122
115 /** 123 /**
116 * Inserts the comments from JSON into a single file. 124 * Inserts the comments from JSON into a single file.
117 */ 125 */
118 void _convertFile(File file, Map<String, List<String>> comments) { 126 void _convertFile(File file, Map<String, List<String>> comments) {
119 var fileLines = file.readAsLinesSync(); 127 var fileLines = file.readAsLinesSync();
(...skipping 22 matching lines...) Expand all
142 150
143 // TODO(amouravski): file.writeAsStringSync('${Strings.join(fileLines, '\n')}\ n'); 151 // TODO(amouravski): file.writeAsStringSync('${Strings.join(fileLines, '\n')}\ n');
144 var outputStream = file.openOutputStream(); 152 var outputStream = file.openOutputStream();
145 outputStream.writeString(Strings.join(fileLines, '\n')); 153 outputStream.writeString(Strings.join(fileLines, '\n'));
146 outputStream.writeString('\n'); 154 outputStream.writeString('\n');
147 155
148 outputStream.onNoPendingWrites = () { 156 outputStream.onNoPendingWrites = () {
149 outputStream.close(); 157 outputStream.close();
150 }; 158 };
151 } 159 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698