This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 36720 - MIMEOptionsFiles should use SAX rather than DOM
Summary: MIMEOptionsFiles should use SAX rather than DOM
Status: RESOLVED WONTFIX
Alias: None
Product: editor
Classification: Unclassified
Component: Settings (show other bugs)
Version: 3.x
Hardware: PC Linux
: P4 blocker (vote)
Assignee: Vitezslav Stejskal
URL:
Keywords: PERFORMANCE
Depends on:
Blocks:
 
Reported: 2003-10-21 14:31 UTC by _ rkubacki
Modified: 2008-01-10 12:48 UTC (History)
1 user (show)

See Also:
Issue Type: DEFECT
Exception Reporter:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description _ rkubacki 2003-10-21 14:31:28 UTC 
SAX should be more effective especially for such
simple format and SAXParser is loaded anyway. Now
editor pays for initialization of DOM builder (in
warmup task).
Comment 1 Martin Roskanin 2004-01-23 12:59:04 UTC 
I am not sure whether I will have enough time to fix it to NB 3.6
because of other priorities
Comment 2 _ rkubacki 2004-09-02 12:55:07 UTC 
This is not only about key binding but I also see some
PropertiessMIMEOptionFile that calls parsing from a AWT thread during
typing (invoked from ExtKit$ExtDefaultKeyTypedAction.actionPerfomed ->
ExtKit$ExtDefaultKeyTypedAction.checkIndentHotChars ->
NbEditorDocument.getFormatter). IMO we should smooth out all these
paths to make typing as fast as possible.
Comment 3 Miloslav Metelka 2004-09-02 18:11:30 UTC 
IMO the reading of those settings only happens once - I guess during
the first keystroke for the particular mime type. They should never
happen again then.
I don't think that there will be enough time to do this now so marking
as "future" for now. We might do it in the 4.0 timeframe but I suspect
that (assuming that the reading is only done once) there are more
serious issues than this one.
Comment 4 Vitezslav Stejskal 2007-02-22 11:21:08 UTC 
MimeOptionsFiles is hopefully going to die with the proper MimeLookup-ization of
editor settings (see issue #90403). The font&color and keybinding settings have
already been updated and their files are parsed using a SAX parser (see
editor/settings/storage).
Comment 5 Jan Lahoda 2007-10-29 17:26:09 UTC 
Seems like P4 to me.
Comment 6 Vitezslav Stejskal 2008-01-10 12:48:25 UTC 
MimeOptionsFiles are deprecated and not used anymore, so closing as WONTFIX. The new editor/settings/storage does use
SAX for reading settings files.