[XUL 1.0] All .dtd files should be renamed to .ent files

RESOLVED WONTFIX

Status

()

defect
RESOLVED WONTFIX
19 years ago
11 years ago

People

(Reporter: hyatt, Assigned: hyatt)

Tracking

Trunk
mozilla1.0
x86
Other
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: WONTFIX ? -- more pain than gain?)

Assignee

Description

19 years ago
This is going to be a big one. :)
Assignee

Updated

19 years ago
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla1.0
Assignee

Updated

19 years ago
Blocks: 70753

Comment 1

19 years ago
Way to go! Minor comments:

  1. Be advised not to rename those in "xmlterm" or the like whose DTDs do 
     contain definition other than text entities.
  2. Is this gonna apply to packaged jar or the source files? My intuition 
     is the latter.

Comment 2

19 years ago
This is going be a big deal for l10n. We have tools, scripts and translation 
memory databases related to the filename and extension like "dtd" and 
"properties". Don't know why we need to change the file extension at this 
moment.
*Why* should .dtd files be renamed to .ent?
This is a fair question. What's the advantage of changing the extension?
Whiteboard: WONTFIX ? -- more pain than gain?

Comment 5

19 years ago
> *Why* should .dtd files be renamed to .ent?

Based on my understanding, our *.dtd holds nothing but text entities (so far), 
and the industry convention is to use ".ent" for such files.

There is nothing wrong with the current extension "*.dtd", it is matter of
following "convention".

BTW, Hyatt:

I was referring to MathML (instead of xmlterm) whose DTD might contain 
definitions other than general entities.

Comment 6

19 years ago
speaking of MathML, I removed all the comments/definitions in the Mozilla's 
version (knowing that Expat wasn't a validating parser). This reduced the size 
of the file quite a bit, from 381K to 64K...

Comment 7

19 years ago
If you make this change it will make our job of localization much harder. Is it
really necessary to "fix" this bug? Is it possible to mark this is "wontfix"?

thanks 
Assignee

Comment 8

19 years ago
Tao and rchen, I will leave this decision to you.  If you do not want to make
the change to .ent, I will close this bug as WONTFIX.  Or if you want to
postpone it to post-Mozilla 1.0, I can do that too.

Comment 9

19 years ago
I can see more pain than gain at this moment. 

It is also arguable that using .dtd to store entity doesn't follow the 
convention. According to XML document in w3c, ENTITY is one of the four basic
keywords in DTD declarations. It is legitimate to have ENTITY in DTD. 

Comment 10

19 years ago
Either way is fine; I don't feel strongly about this. I'll leave this to L10n
group.

Comment 11

19 years ago
Thank you for being open minded and considerate.
Assignee

Comment 12

19 years ago
Ok, resolving WONTFIX.
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → WONTFIX
woohoo! VERIFIED WONTFIX
Status: RESOLVED → VERIFIED

Comment 14

19 years ago
reopening for reconsideration. I was just looking at today's bugs and saw
"bug 75188 - mailEditorOverlay.dtd redundant?" and upon clicking, it just
seemed inelegant to keep calling these things ".dtd". It is like calling your
".h" files ".cpp"...  There have been successful jihads to cleanup such things
in a very efficient way. I still have the script that I used for bug 7834...
Status: VERIFIED → REOPENED
Resolution: WONTFIX → ---

Comment 15

19 years ago
We have decided as a team not to fix this one because of the problems it will
present to the localization process. As you know we must ship this product
simultaneously in 3 languages. To do so we need to leave this one alone.
Remarking it as "wontfix".
Status: REOPENED → RESOLVED
Closed: 19 years ago19 years ago
Resolution: --- → WONTFIX

Comment 16

19 years ago
Reopening because I don't have the right to mark it "wontfix". Sorry about that 
Hyatt. If you'd like please remark it "wontfix". thanks.
Status: RESOLVED → REOPENED
Resolution: WONTFIX → ---

Comment 17

19 years ago
It's completely up to you, this is your team's domain.  Remarking...
Status: REOPENED → RESOLVED
Closed: 19 years ago19 years ago
Resolution: --- → WONTFIX

Updated

11 years ago
Component: XP Toolkit/Widgets: XUL → XUL
QA Contact: jrgmorrison → xptoolkit.widgets
You need to log in before you can comment on or make changes to this bug.