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

Side by Side Diff: pkg/analysis_server/lib/src/analysis_server.dart

Issue 652903006: Issue 21373. Fix for removing line info in case of exception (causes NPE in outline computer). (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 6 years, 2 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | pkg/analysis_server/lib/src/computer/computer_outline.dart » ('j') | 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) 2014, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2014, 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 library analysis.server; 5 library analysis.server;
6 6
7 import 'dart:async'; 7 import 'dart:async';
8 import 'dart:collection'; 8 import 'dart:collection';
9 9
10 import 'package:analyzer/file_system/file_system.dart'; 10 import 'package:analyzer/file_system/file_system.dart';
(...skipping 633 matching lines...) Expand 10 before | Expand all | Expand 10 after
644 sendAnalysisNotificationHighlights(this, file, dartUnit); 644 sendAnalysisNotificationHighlights(this, file, dartUnit);
645 break; 645 break;
646 case AnalysisService.NAVIGATION: 646 case AnalysisService.NAVIGATION:
647 // TODO(scheglov) consider support for one unit in 2+ libraries 647 // TODO(scheglov) consider support for one unit in 2+ libraries
648 sendAnalysisNotificationNavigation(this, file, dartUnit); 648 sendAnalysisNotificationNavigation(this, file, dartUnit);
649 break; 649 break;
650 case AnalysisService.OCCURRENCES: 650 case AnalysisService.OCCURRENCES:
651 sendAnalysisNotificationOccurrences(this, file, dartUnit); 651 sendAnalysisNotificationOccurrences(this, file, dartUnit);
652 break; 652 break;
653 case AnalysisService.OUTLINE: 653 case AnalysisService.OUTLINE:
654 LineInfo lineInfo = context.getLineInfo(source);
654 sendAnalysisNotificationOutline( 655 sendAnalysisNotificationOutline(
655 this, 656 this,
656 context,
657 source, 657 source,
658 lineInfo,
658 dartUnit); 659 dartUnit);
659 break; 660 break;
660 case AnalysisService.OVERRIDES: 661 case AnalysisService.OVERRIDES:
661 sendAnalysisNotificationOverrides(this, file, dartUnit); 662 sendAnalysisNotificationOverrides(this, file, dartUnit);
662 break; 663 break;
663 } 664 }
664 } 665 }
665 } 666 }
666 } 667 }
667 }); 668 });
(...skipping 265 matching lines...) Expand 10 before | Expand all | Expand 10 after
933 // send the notification 934 // send the notification
934 channel.sendNotification( 935 channel.sendNotification(
935 new ServerErrorParams( 936 new ServerErrorParams(
936 true, 937 true,
937 exceptionString, 938 exceptionString,
938 stackTraceString).toNotification()); 939 stackTraceString).toNotification());
939 } 940 }
940 } 941 }
941 942
942 typedef void OptionUpdater(AnalysisOptionsImpl options); 943 typedef void OptionUpdater(AnalysisOptionsImpl options);
OLDNEW
« no previous file with comments | « no previous file | pkg/analysis_server/lib/src/computer/computer_outline.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698