58516 - javascript strict warnings in builtinURLs.js

Status: VERIFIED FIXED

(SeaMonkey :: General, defect, P3)

Description

[Blake Ross]

redeclaration of var ds_uri

Comment 1

[Blake Ross]

Attachment: patch

Comment 2

[timeless]

Erm, 
87 var ds_uri = "chrome://global/locale/builtinURLs.rdf";
88 var url_ds = gRDFService.GetDataSource(ds_uri); // return nsIRDFDataSource
ds_uri, url_ds? What style are we using here? Ben: this is your code...

blake: r=timeless, but i don't like the original code, and I think it would be 
better if it were cleaned.

Comment 3

[Doron Rosenberg (IBM)]

keywords

Comment 4

[Brendan Eich [:brendan]]

Wow, tabs and other problems -- I don't think that's ben's doing.  More likely 
to be rjc's.  Give this file a facelift, I'll review and ben or alecf can sr=.

/be

Comment 5

[Blake Ross]

*** Bug 58563 has been marked as a duplicate of this bug. ***

Comment 6

[Blake Ross]

alec's dnd fix fixed this.

Comment 7

[Henrik Gemal]

verified with build 20010126 on win2k

Comment 8

[Henrik Gemal]

sorry for the spam.... just changing the subject so that the bugs can be sorted 
correctly