Login | Register For Free | Help
Search for: (Advanced)

Mailing List Archive: Python: Bugs

[issue11379] Remove "lightweight" from minidom description

 

 

First page Previous page 1 2 Next page Last page  View All Python bugs RSS feed   Index | Next | Previous | View Threaded


report at bugs

Feb 29, 2012, 4:40 AM

Post #1 of 26 (314 views)
Permalink
[issue11379] Remove "lightweight" from minidom description

Changes by Ezio Melotti <ezio.melotti [at] gmail>:


----------
stage: -> needs patch

_______________________________________
Python tracker <report [at] bugs>
<http://bugs.python.org/issue11379>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com


report at bugs

Feb 29, 2012, 8:37 AM

Post #2 of 26 (308 views)
Permalink
[issue11379] Remove "lightweight" from minidom description [In reply to]

Martin v. Löwis <martin [at] v> added the comment:

The wording in msg152836 is fine with me, in particular as it doesn't make any performance claims.

----------

_______________________________________
Python tracker <report [at] bugs>
<http://bugs.python.org/issue11379>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com


report at bugs

Feb 29, 2012, 11:04 AM

Post #3 of 26 (309 views)
Permalink
[issue11379] Remove "lightweight" from minidom description [In reply to]

Eli Bendersky <eliben [at] gmail> added the comment:

I'm attaching a patch for Doc/library/xml.dom.minidom.rst

It adds the note as phrased by Stefan, with a tiny wording change to make the first sentence less ambiguous.

----------
keywords: +patch
Added file: http://bugs.python.org/file24686/issue_11379.1.patch

_______________________________________
Python tracker <report [at] bugs>
<http://bugs.python.org/issue11379>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com


report at bugs

Feb 29, 2012, 9:23 PM

Post #4 of 26 (309 views)
Permalink
[issue11379] Remove "lightweight" from minidom description [In reply to]

Éric Araujo <merwok [at] netwok> added the comment:

I’m not sure I would use note markup, though (cf. Raymond’s aversion to littering the doc with note and warning boxes).

----------

_______________________________________
Python tracker <report [at] bugs>
<http://bugs.python.org/issue11379>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com


report at bugs

Feb 29, 2012, 10:33 PM

Post #5 of 26 (308 views)
Permalink
[issue11379] Remove "lightweight" from minidom description [In reply to]

Eli Bendersky <eliben [at] gmail> added the comment:

>
> I’m not sure I would use note markup, though (cf. Raymond’s aversion to
> littering the doc with note and warning boxes).
>

I also dislike box littering, but this one seems like a really good fit for
a note, since it's completely outside the flow of that documentation page.

----------

_______________________________________
Python tracker <report [at] bugs>
<http://bugs.python.org/issue11379>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com


report at bugs

Feb 29, 2012, 11:04 PM

Post #6 of 26 (308 views)
Permalink
[issue11379] Remove "lightweight" from minidom description [In reply to]

Raymond Hettinger <raymond.hettinger [at] gmail> added the comment:

This is a reasonable case for a note.

----------
nosy: +rhettinger

_______________________________________
Python tracker <report [at] bugs>
<http://bugs.python.org/issue11379>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com


report at bugs

Mar 1, 2012, 9:44 PM

Post #7 of 26 (311 views)
Permalink
[issue11379] Remove "lightweight" from minidom description [In reply to]

Roundup Robot <devnull [at] psf> added the comment:

New changeset 81e606862a89 by Eli Bendersky in branch '3.2':
Issue #11379: add a note in xml.dom.minidom suggesting to use etree in some cases
http://hg.python.org/cpython/rev/81e606862a89

----------
nosy: +python-dev

_______________________________________
Python tracker <report [at] bugs>
<http://bugs.python.org/issue11379>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com


report at bugs

Mar 1, 2012, 9:46 PM

Post #8 of 26 (309 views)
Permalink
[issue11379] Remove "lightweight" from minidom description [In reply to]

Roundup Robot <devnull [at] psf> added the comment:

New changeset ccd16ad37544 by Eli Bendersky in branch '2.7':
Issue #11379: add a note in xml.dom.minidom suggesting to use etree in some cases
http://hg.python.org/cpython/rev/ccd16ad37544

----------

_______________________________________
Python tracker <report [at] bugs>
<http://bugs.python.org/issue11379>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com


report at bugs

Mar 1, 2012, 9:47 PM

Post #9 of 26 (308 views)
Permalink
[issue11379] Remove "lightweight" from minidom description [In reply to]

Eli Bendersky <eliben [at] gmail> added the comment:

Committed to 2.7, 3.2 and 3.3

I suppose this issue can be closed now?

----------

_______________________________________
Python tracker <report [at] bugs>
<http://bugs.python.org/issue11379>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com


report at bugs

Mar 2, 2012, 1:54 AM

Post #10 of 26 (309 views)
Permalink
[issue11379] Remove "lightweight" from minidom description [In reply to]

Stefan Behnel <scoder [at] users> added the comment:

Thanks Eli.

What about the "Lightweight DOM implementation", though? Following Martin's comment that performance characteristics (like "fast", "memory friendly" or "lightweight") should normally not be documented, I'm still suggesting to replace it with a less easily misinterpreted phrase like "W3C DOM implementation".

----------

_______________________________________
Python tracker <report [at] bugs>
<http://bugs.python.org/issue11379>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com


report at bugs

Mar 2, 2012, 2:46 AM

Post #11 of 26 (308 views)
Permalink
[issue11379] Remove "lightweight" from minidom description [In reply to]

Eli Bendersky <eliben [at] gmail> added the comment:

Stefan, frankly I'm not familiar enough with either xml.dom or xml.dom.minidom to have a solid opinion at this point.

----------

_______________________________________
Python tracker <report [at] bugs>
<http://bugs.python.org/issue11379>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com


report at bugs

Mar 2, 2012, 2:51 AM

Post #12 of 26 (307 views)
Permalink
[issue11379] Remove "lightweight" from minidom description [In reply to]

Éric Araujo <merwok [at] netwok> added the comment:

I think I’ve always understood “lightweight” to mean “minimal”. xml.dom provides minidom, a basic implementation, pulldom, a different implementation, and other libraries such as 4Dom are full-fledged implementations. So “lightweight” is not a problem to me (but I acknowledge that it might be misleading for other people), especially given that I think that DOM itself is not elegant or lightweight (as in “conceptually small”).

----------

_______________________________________
Python tracker <report [at] bugs>
<http://bugs.python.org/issue11379>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com


report at bugs

Mar 2, 2012, 6:51 AM

Post #13 of 26 (309 views)
Permalink
[issue11379] Remove "lightweight" from minidom description [In reply to]

Antoine Pitrou <pitrou [at] free> added the comment:

> I think I’ve always understood “lightweight” to mean “minimal”.

Then how about saying "minimal" instead of "lightweight"?
(also, it seems it really means "incomplete" or "partial", which are of course less positive sounding)

----------

_______________________________________
Python tracker <report [at] bugs>
<http://bugs.python.org/issue11379>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com


report at bugs

Mar 2, 2012, 6:58 AM

Post #14 of 26 (308 views)
Permalink
[issue11379] Remove "lightweight" from minidom description [In reply to]

Ezio Melotti <ezio.melotti [at] gmail> added the comment:

"Minimal" sounds good to me, it also matches the name of the module.

----------

_______________________________________
Python tracker <report [at] bugs>
<http://bugs.python.org/issue11379>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com


report at bugs

Mar 2, 2012, 7:40 AM

Post #15 of 26 (308 views)
Permalink
[issue11379] Remove "lightweight" from minidom description [In reply to]

Éric Araujo <merwok [at] netwok> added the comment:

Right, patch for 3.2. Also edited the module docstring (info taken from the docstring of xml.dom). BTW I really think we could have avoided some verbosity by adding the recommendation to use xml.etree in the first paragraph of Doc/library/xml.dom.minidom.rst.

----------
Added file: http://bugs.python.org/file24707/minidom-desc.diff

_______________________________________
Python tracker <report [at] bugs>
<http://bugs.python.org/issue11379>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com


report at bugs

Mar 2, 2012, 7:41 AM

Post #16 of 26 (310 views)
Permalink
[issue11379] Remove "lightweight" from minidom description [In reply to]

Éric Araujo <merwok [at] netwok> added the comment:

s/Mininal/Minimal/ in the synopsis

----------

_______________________________________
Python tracker <report [at] bugs>
<http://bugs.python.org/issue11379>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com


report at bugs

Mar 2, 2012, 7:55 AM

Post #17 of 26 (309 views)
Permalink
[issue11379] Remove "lightweight" from minidom description [In reply to]

Stefan Behnel <scoder [at] users> added the comment:

Yes, I think that's better.

----------

_______________________________________
Python tracker <report [at] bugs>
<http://bugs.python.org/issue11379>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com


report at bugs

Mar 4, 2012, 9:21 PM

Post #18 of 26 (287 views)
Permalink
[issue11379] Remove "lightweight" from minidom description [In reply to]

Éric Araujo <merwok [at] netwok> added the comment:

This alternate version of my patch (a) merges the first two paragraphs to make the intro less redundant and heavy, and (b) reorganizes a bit the list of modules in Doc/library/markup.rst to have xml.etree first and pyexpat (less interesting for most people) at the end. Tell me if you prefer this version, or if I should commit the first one (possibly with the (b) change).

----------
Added file: http://bugs.python.org/file24732/minidom-desc-2.diff

_______________________________________
Python tracker <report [at] bugs>
<http://bugs.python.org/issue11379>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com


report at bugs

Mar 5, 2012, 7:26 AM

Post #19 of 26 (288 views)
Permalink
[issue11379] Remove "lightweight" from minidom description [In reply to]

Roundup Robot <devnull [at] psf> added the comment:

New changeset d99c0a4b66f3 by Éric Araujo in branch '3.2':
Move xml.etree higher and xml.parsers.expat lower in the markup ToC.
http://hg.python.org/cpython/rev/d99c0a4b66f3

----------

_______________________________________
Python tracker <report [at] bugs>
<http://bugs.python.org/issue11379>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com


report at bugs

Mar 5, 2012, 7:48 AM

Post #20 of 26 (287 views)
Permalink
[issue11379] Remove "lightweight" from minidom description [In reply to]

Roundup Robot <devnull [at] psf> added the comment:

New changeset fc32753feb0a by Éric Araujo in branch '2.7':
Move xml.etree higher and xml.parsers.expat lower in the markup ToC.
http://hg.python.org/cpython/rev/fc32753feb0a

----------

_______________________________________
Python tracker <report [at] bugs>
<http://bugs.python.org/issue11379>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com


report at bugs

Mar 7, 2012, 3:37 AM

Post #21 of 26 (286 views)
Permalink
[issue11379] Remove "lightweight" from minidom description [In reply to]

Éric Araujo <merwok [at] netwok> added the comment:

FYI, note that http://wiki.python.org/moin/MiniDom says this about minidom: “slow and very memory hungry DOM implementation”.

As you have seen, I have applied my ToC order change. Now in order to commit my s/lightweight/minimal/ change and close this report, can you Eli say if minidom-desc-2 is okay (I’m asking you because this patch touches text you just added, contrary to minidom-desc)?

----------

_______________________________________
Python tracker <report [at] bugs>
<http://bugs.python.org/issue11379>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com


report at bugs

Mar 7, 2012, 3:44 AM

Post #22 of 26 (286 views)
Permalink
[issue11379] Remove "lightweight" from minidom description [In reply to]

Martin v. Löwis <martin [at] v> added the comment:

> FYI, note that http://wiki.python.org/moin/MiniDom says this about
> minidom: “slow and very memory hungry DOM implementation”.

Thanks for the notice; I have now fixed that wording.

----------

_______________________________________
Python tracker <report [at] bugs>
<http://bugs.python.org/issue11379>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com


report at bugs

Mar 7, 2012, 3:53 AM

Post #23 of 26 (286 views)
Permalink
[issue11379] Remove "lightweight" from minidom description [In reply to]

Eli Bendersky <eliben [at] gmail> added the comment:

Éric,

I'm ok with replacing "lightweight" by "minimal", unless others have objections. Regarding the specifics of the minidom-desc-2.diff patch:

"proficient with the DOM"

I'm not sure "the DOM" is semantically correct. "the W3C-DOM interface" is more precise.

Also, I still think that a note would be more appropriate, but I don't care enough to argue about it :)

----------

_______________________________________
Python tracker <report [at] bugs>
<http://bugs.python.org/issue11379>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com


report at bugs

Mar 7, 2012, 4:27 AM

Post #24 of 26 (289 views)
Permalink
[issue11379] Remove "lightweight" from minidom description [In reply to]

Stefan Behnel <scoder [at] users> added the comment:

Oh, right, I missed that part. I also think that a visible note is better. And +1 for "W3C DOM interface".

----------

_______________________________________
Python tracker <report [at] bugs>
<http://bugs.python.org/issue11379>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com


report at bugs

Mar 16, 2012, 5:57 AM

Post #25 of 26 (258 views)
Permalink
[issue11379] Remove "lightweight" from minidom description [In reply to]

Eli Bendersky <eliben [at] gmail> added the comment:

Éric, what else would you like to do here?

----------

_______________________________________
Python tracker <report [at] bugs>
<http://bugs.python.org/issue11379>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com

First page Previous page 1 2 Next page Last page  View All Python bugs RSS feed   Index | Next | Previous | View Threaded
 
 


Interested in having your list archived? Contact Gossamer Threads
 
  Web Applications & Managed Hosting Powered by Gossamer Threads Inc.