Closed Bug 64445 Opened 24 years ago Closed 24 years ago

0[91d5eb0]: ###!!! ASSERTION: nsScriptNameSetRegistry not thread-safe: 'owningTh

Categories

(Core Graveyard :: Java APIs to WebShell, defect)

x86
Windows NT
defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: edburns, Assigned: edburns)

Details

Attachments

(2 files)

      No description provided.
Posted to n.p.m.embedding:

Your message has been submitted to netscape.public.mozilla.embedding. 

NOTE: If you made a mistake and need to cancel your message, you can cancel it 
from our cancel page.
To do so you will need this Message-ID: <935mes$fr3$1@nnrp1.deja.com>. 

Status: NEW → ASSIGNED
The reason for the assertion is that Preferences doesn't use ActionEvents.

I'm about to attach a checkpoint of the conversion of Prefs to actionEvents.
Attached file tar.gz checkpoint.
Comments about Patch -
* Makefile.win and Makefile.in - ok
* PreferenceImpl.cpp -
  you have code like - goto NSUP_CLEANUP everywhere in this file.
  Instead of goto, you could define a macro for doing the cleanup + return
  from function. Then you can just call that macro eveywhere instead of
  using goto.  
* PreferenceActionEvents.cpp - same problem. goto should be avoided wherever
  possible.
* PreferenceActionEvents.h - ok

pls let me know if you agree with these comments. 
_Ashu
Ashu, I prefer to have a single return in each method, more so than a simple 
local goto.  In any case, this goto logic is used all over the code.  If we're 
going to change that strategy, we should do it everywhere.  Given that, can I 
please have r=?

Thanks,

Ed
r=ashuk

Fix checked in.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Verified with mozilla branch NETSCAPE_6_2_RELEASE and blackwood sources from 
trunk. Preferences works fine, I cannot see any assertions.
Changed QAContact and add myself to cc list.
Status: RESOLVED → VERIFIED
QA Contact: geetha.vaidyanaathan → avm
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: