294989 - In <editorInlineSpellCheck.js>, 4 "Warning: anonymous function does not always return a value"

Status: RESOLVED FIXED

(Core :: Spelling checker, defect)

Description

[Serge Gautherie (:sgautherie)]

[Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.8b2) Gecko/20050519] (nightly) (W98SE)

{{
Warning: anonymous function does not always return a value
Source File: chrome://editor/content/editorInlineSpellCheck.js
Line: 65, Column: 2
Source Code:
  },

Warning: anonymous function does not always return a value
Source File: chrome://editor/content/editorInlineSpellCheck.js
Line: 143, Column: 2
Source Code:
  },

Warning: anonymous function does not always return a value
Source File: chrome://editor/content/editorInlineSpellCheck.js
Line: 159, Column: 2
Source Code:
  },

Warning: anonymous function does not always return a value
Source File: chrome://editor/content/editorInlineSpellCheck.js
Line: 176, Column: 2
Source Code:
  }


}}

Comment 1

[Karsten Düsterloh]

Attachment: trivially remove all unused return values [Checked in: Comment 4]

All these return values are unused and can be removed. Taking.

Comment 2

[Dan Mosedale (:dmosedale, :dmose)]

Comment on attachment 186968 [details] [diff] [review]
trivially remove all unused return values
[Checked in: Comment 4]

sr=dmose

Comment 3

[Karsten Düsterloh]

Comment on attachment 186968 [details] [diff] [review]
trivially remove all unused return values
[Checked in: Comment 4]

trivial strict warning removal

Comment 4

[Karsten Düsterloh]

Checking in editorInlineSpellCheck.js;
new revision: 1.2; previous revision: 1.1
done

Comment 5

[Karsten Düsterloh]

Comment on attachment 186968 [details] [diff] [review]
trivially remove all unused return values
[Checked in: Comment 4]

Serge, please *stop* marking patches as obsolete if they never were that. Just
because they're checked in doesn't mean they're obsolete...