235498 - In <addressingWidgetOverlay.js>, 2 warnings

Status: RESOLVED FIXED

(MailNews Core :: Composition, defect)

Description

[Serge Gautherie (:sgautherie)]

[Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.7a) Gecko/20040219] (W98SE)

Part A
{{
Warning: anonymous function does not always return a value
Source File: chrome://messenger/content/messengercompose/addressingWidgetOverlay.js
Line: 1148, Column: 2
Source Code:
  },
}}

Part B
{{
Warning: redeclaration of var searchResultsForSession
Source File: chrome://messenger/content/messengercompose/addressingWidgetOverlay.js
Line: 1190, Column: 12
Source Code:
        var searchResultsForSession = this.searchResults[sessionIndex];
}}

I believe that they happened when I opened Compose !!?

Comment 1

[Serge Gautherie (:sgautherie)]

Attachment: (Bv1) <addressingWidgetOverlay.js> [Checked in: Comment 5]

Comment 2

[Serge Gautherie (:sgautherie)]

Comment on attachment 142191 [details] [diff] [review]
(Bv1) <addressingWidgetOverlay.js>
[Checked in: Comment 5]


I'm not sure what the correct fix is for Part A:
helpwanted.

Comment 3

[[e-mail]]

Comment on attachment 142191 [details] [diff] [review]
(Bv1) <addressingWidgetOverlay.js>
[Checked in: Comment 5]

I'd like to say that the answer to your other question is to rewrite the code
:-P

Comment 4

[Serge Gautherie (:sgautherie)]

Attachment: (Av1) <addressingWidgetOverlay.js> [Checked in: Comment 6]

Comment 5

[Serge Gautherie (:sgautherie)]

Comment on attachment 142191 [details] [diff] [review]
(Bv1) <addressingWidgetOverlay.js>
[Checked in: Comment 5]


Check in: { 03/05/2004 14:45	neil%parkwaycc.co.uk	mozilla/ mailnews/
compose/ resources/ content/ addressingWidgetOverlay.js      1.85 }

Comment 6

[Serge Gautherie (:sgautherie)]

Comment on attachment 142287 [details] [diff] [review]
(Av1) <addressingWidgetOverlay.js>
[Checked in: Comment 6]


Check in: { 03/09/2004 16:23	neil%parkwaycc.co.uk	mozilla/ mailnews/
compose/ resources/ content/ addressingWidgetOverlay.js      1.86 }