Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 466862

Summary:	[eslint] missing-nls rule should ignore literals used in case statements
Product:	[ECD] Orion	Reporter:	Michael Rennie <Michael_Rennie>
Component:	JS Tools	Assignee:	Michael Rennie <Michael_Rennie>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	P3
Version:	8.0
Target Milestone:	9.0
Hardware:	All
OS:	All
Whiteboard:

Description Michael Rennie

2015-05-08 12:14:37 EDT

Consider the following snippet:

var c = 'b';
switch(c) {
	case 'a':{}
	case 'b': {}
}

'a' and 'b' are marked, but they shouldn't be

Comment 1 Michael Rennie

2015-05-08 14:12:26 EDT

Fix + new tests:

http://git.eclipse.org/c/orion/org.eclipse.orion.client.git/commit/?id=deba31979b02653c0bf5eeec65dccc75843a2716

I also updated skipping for importScripts, Worker, SharedWorker, require([]), define(id) and requirejs.