From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Chong Yidong Newsgroups: gmane.emacs.devel Subject: Re: Problems with xml-parse-string Date: Wed, 22 Sep 2010 10:05:27 -0400 Message-ID: <87zkv97u1k.fsf@stupidchicken.com> References: <87pqw6d7nz.fsf@stupidchicken.com> <87zkvaiked.fsf@stupidchicken.com> <87vd5ymptn.fsf@stupidchicken.com> NNTP-Posting-Host: lo.gmane.org Mime-Version: 1.0 Content-Type: text/plain X-Trace: dough.gmane.org 1285164355 8060 80.91.229.12 (22 Sep 2010 14:05:55 GMT) X-Complaints-To: usenet@dough.gmane.org NNTP-Posting-Date: Wed, 22 Sep 2010 14:05:55 +0000 (UTC) To: emacs-devel@gnu.org Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Wed Sep 22 16:05:53 2010 Return-path: Envelope-to: ged-emacs-devel@m.gmane.org Original-Received: from lists.gnu.org ([199.232.76.165]) by lo.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1OyPx8-0005VS-Sz for ged-emacs-devel@m.gmane.org; Wed, 22 Sep 2010 16:05:43 +0200 Original-Received: from localhost ([127.0.0.1]:58932 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1OyPx8-0000JV-A1 for ged-emacs-devel@m.gmane.org; Wed, 22 Sep 2010 10:05:42 -0400 Original-Received: from [140.186.70.92] (port=42189 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1OyPx0-0000Ij-W0 for emacs-devel@gnu.org; Wed, 22 Sep 2010 10:05:36 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69) (envelope-from ) id 1OyPwv-0002qN-Ou for emacs-devel@gnu.org; Wed, 22 Sep 2010 10:05:34 -0400 Original-Received: from pantheon-po43.its.yale.edu ([130.132.50.104]:51467) by eggs.gnu.org with esmtp (Exim 4.69) (envelope-from ) id 1OyPwv-0002qE-NL for emacs-devel@gnu.org; Wed, 22 Sep 2010 10:05:29 -0400 Original-Received: from furry (adsl-99-103-105-16.dsl.wlfrct.sbcglobal.net [99.103.105.16]) (authenticated bits=0) by pantheon-po43.its.yale.edu (8.12.11.20060308/8.12.11) with ESMTP id o8ME5RPW006235 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NOT) for ; Wed, 22 Sep 2010 10:05:28 -0400 Original-Received: by furry (Postfix, from userid 1000) id 3D46316D402; Wed, 22 Sep 2010 10:05:27 -0400 (EDT) In-Reply-To: (Lars Magne Ingebrigtsen's message of "Wed, 22 Sep 2010 12:35:49 +0200") User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.0.50 (gnu/linux) X-YaleITSMailFilter: Version 1.2c (attachment(s) not renamed) X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.4-2.6 X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Original-Sender: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.devel:130621 Archived-At: Lars Magne Ingebrigtsen writes: > Euhm. Actually, I spent quite a bit of thought on the parse tree format > that the functions spit out to make it regular, easy and fast to deal > with. The format that xml.el spits out is more arcane and less > comfortable to use. You can't say `(assq 'img (cdr node))' and stuff to > get the image nodes, because the child nodes aren't a regular assoc > list (because of the string fields) and stuff. > > So I protest the change. > > If you're concerned about previous users of xml.el being confused by the > change in parse tree format, I don't think you have to worry all that > much, because there are zero (0) users of the xml.el `xml-parse-string' > in the Emacs tree. The issue isn't xml-parse-string, it's xml-parse-region and xml-parse-file, which are used by several parts of Emacs including, ahem, Gnus. I don't want to have two parts of Emacs providing xml parsing that provide slightly incompatible parse tree formats. So either the new libxml functions have to provide the same format as xml.el, or xml.el has to be changed to used the new format, breaking existing uses. I am amenable to the latter if the new format is so much better than the old one that it's worth dealing with the backward compatibility headaches.