javascript strict warnings in msgViewNavigation.js

VERIFIED WORKSFORME

Status

SeaMonkey
MailNews: Message Display
P3
normal
VERIFIED WORKSFORME
18 years ago
13 years ago

People

(Reporter: Henrik Gemal, Assigned: Peter ``jag'' Annema)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

18 years ago
JavaScript strict warning:
chrome://messenger/content/msgViewNavigation.js line 40: function 
FindNextFolder does not always return a value

JavaScript strict warning:
chrome://messenger/content/msgViewNavigation.js line 52: function 
FindNextFolder does not always return a value

JavaScript strict warning:
chrome://messenger/content/msgViewNavigation.js line 56: function 
FindNextFolder does not always return a value

JavaScript strict warning:
chrome://messenger/content/msgViewNavigation.js line 73: function 
FindNextFolder does not always return a value

JavaScript strict warning:
chrome://messenger/content/msgViewNavigation.js line 78: function 
FindNextFolder does not always return a value
(Assignee)

Comment 1

17 years ago
putterman: shall I take this?
(Assignee)

Comment 2

17 years ago
Created attachment 21107 [details] [diff] [review]
[patch] fix warnings
(Assignee)

Comment 3

17 years ago
Created attachment 21108 [details] [diff] [review]
[patch] fix warnings
(Assignee)

Comment 4

17 years ago
Heh, oops, clicked the button twice.

Comment 5

17 years ago
jag, sorry, I'm just catching up on my bugs. Thanks for providing a patch.  
reassigning to you since you have the patch and cc'ing sspitzer for review since 
I think he wrote the original code for FindNextFolder
Assignee: putterman → disttsc
Keywords: patch
r=sspitzer

thanks for catching my mistake, jag.
(Assignee)

Comment 7

17 years ago
Who can a= this?
david, can you sr= jag's patch?  I've already reviewed it.

Comment 9

17 years ago
sr=bienvenu
(Assignee)

Updated

17 years ago
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
(Assignee)

Comment 10

17 years ago
Checked in, marking fixed.

Comment 11

17 years ago
Verified Linux 20010106
Status: RESOLVED → VERIFIED
(Reporter)

Updated

17 years ago
Status: VERIFIED → REOPENED
Resolution: FIXED → ---
(Reporter)

Comment 12

17 years ago
After the view mail front end has landed:

JavaScript strict warning:
chrome://messenger/content/msgViewNavigation.js line 107: function 
CrossFolderNavigation does not always return a value

JavaScript strict warning:
chrome://messenger/content/msgViewNavigation.js line 171: function 
CrossFolderNavigation does not always return a value

JavaScript strict warning:
chrome://messenger/content/msgMail3PaneWindow.js line 206: redeclaration of var
outlinerView

Comment 13

17 years ago
I don't see these warnings lately, and the code seems to be fixed now.

Marking worksforme.
Status: REOPENED → RESOLVED
Last Resolved: 17 years ago17 years ago
Resolution: --- → WORKSFORME
QA Contact: esther → stephend
I didn't see these either with build 2001-05-31-04 on win2k.
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.