18091 - uninitialized variable 's' used

Status: VERIFIED FIXED

(SeaMonkey :: General, defect, P3)

Description

[Brendan Eich [:brendan]]

nsPICSElementObserver.cpp(195) : warning C4700: local variable 's' used without
having been initialized

Isn't extensions/pics going to lose files that are not licensed compatibly?  Who
will do the work?  Cc'ing shaver and leaf.

/be

Comment 1

[Daniel (Leaf) Nunes]

I was going to remove the PICS code from the mozilla tree, because almost all of
the code is W3C licensed, and shaver and mitchell have stated that it's not
compatible. dveditz seemed to contend this point, but i haven't heard
resolution, so i'll assume that mitchell and shaver are sticking to their guns
on this, and i'll remove the PICS directory forthwith.

Comment 2

[Daniel Bratell]

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

Comment 3

[Mike Shaver (:shaver -- probably not reading bugmail closely)]

Leaf?  Can you just remove this directory instead of having it latered to
infinity and beyond?

Comment 4

[Daniel (Leaf) Nunes]

at last, the evil has been purged. pics has been out of the build for a long
time, just cvs removed the pics/src/ source files.

Comment 5

[Paul MacQuiddy]

ahhh, it's like pulling a bad tooth