From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!.POSTED.blaine.gmane.org!not-for-mail From: Lars Ingebrigtsen Newsgroups: gmane.emacs.bugs Subject: bug#32359: [PATCH] Add svg-path Date: Fri, 12 Jul 2019 18:02:56 +0200 Message-ID: References: <83zhy3txbv.fsf@gnu.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Injection-Info: blaine.gmane.org; posting-host="blaine.gmane.org:195.159.176.226"; logging-data="134399"; mail-complaints-to="usenet@blaine.gmane.org" User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/27.0.50 (gnu/linux) Cc: 32359@debbugs.gnu.org To: "Felix E. Klee" Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Fri Jul 12 18:04:08 2019 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane.org Original-Received: from lists.gnu.org ([209.51.188.17]) by blaine.gmane.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.89) (envelope-from ) id 1hly1o-000Ypz-Io for geb-bug-gnu-emacs@m.gmane.org; Fri, 12 Jul 2019 18:04:08 +0200 Original-Received: from localhost ([::1]:51004 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hly1n-0007aT-Ju for geb-bug-gnu-emacs@m.gmane.org; Fri, 12 Jul 2019 12:04:07 -0400 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]:42792) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hly1k-0007aB-5Z for bug-gnu-emacs@gnu.org; Fri, 12 Jul 2019 12:04:05 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1hly1i-00030i-Lp for bug-gnu-emacs@gnu.org; Fri, 12 Jul 2019 12:04:04 -0400 Original-Received: from debbugs.gnu.org ([209.51.188.43]:59996) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1hly1i-0002zb-I0 for bug-gnu-emacs@gnu.org; Fri, 12 Jul 2019 12:04:02 -0400 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1hly1i-00067T-2d for bug-gnu-emacs@gnu.org; Fri, 12 Jul 2019 12:04:02 -0400 X-Loop: help-debbugs@gnu.org Resent-From: Lars Ingebrigtsen Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Fri, 12 Jul 2019 16:04:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 32359 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: patch Original-Received: via spool by 32359-submit@debbugs.gnu.org id=B32359.156294738223452 (code B ref 32359); Fri, 12 Jul 2019 16:04:02 +0000 Original-Received: (at 32359) by debbugs.gnu.org; 12 Jul 2019 16:03:02 +0000 Original-Received: from localhost ([127.0.0.1]:40584 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1hly0j-000668-Qa for submit@debbugs.gnu.org; Fri, 12 Jul 2019 12:03:02 -0400 Original-Received: from quimby.gnus.org ([80.91.231.51]:46486) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1hly0i-00065l-DN for 32359@debbugs.gnu.org; Fri, 12 Jul 2019 12:03:01 -0400 Original-Received: from cm-84.212.202.86.getinternet.no ([84.212.202.86] helo=stories) by quimby.gnus.org with esmtpsa (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.89) (envelope-from ) id 1hly0e-0007bN-Ln; Fri, 12 Jul 2019 18:02:58 +0200 Face: iVBORw0KGgoAAAANSUhEUgAAADAAAAAwBAMAAAClLOS0AAAAD1BMVEUMNF0YU3kEI0sVQms5 gaLX3xRUAAACS0lEQVQ4jV1TAZIjIQhEnQdI9AHG7AM0zAPCyf/fdI2TzW4tVakoLQ00DLW+LShv q7fr3unjn8wigJ6/gff7uf/iD/BQfwceP7NGcsO5Xa++OO/ounq7gHBFN77Bg6vlK+LB70o4+wOK rBcw+fI3u7m7U9iH/niXTlThjA1QZeecevljPxG/k9711qjV/Pb3M+7kIK2FKOxWG/zt3H9uJ9Kz M7WBHx3tLVAL+iIu7/6pf423gJGMyZvYRL0f/VtZqkb6vAJiT6HJ2FNodBpBjh2AHk6mmfE8iQCY 3xlaqqVRhU6QMFSaDSVFtBHvRy3M+kTzlJjKbvYR/ZVCioM9ghZhbKpsWqueLueDt+gLhX1sBoaI mR1geo2SJJUpQ+ZgbIOeIjKVXtDRSb0UmnDZzCSihMbj17VcqTf4qqlij+gfVoQbEGoSu8tdrZpH 3HPeSvU2ISu1I7zMllIxQ4LxeEaBCIFoptdhC32ojk4D4hd0MVFoWsl00dQVY5u5pcJogqiERaaF DgAYMrUgR+FBIQcls4IlWoMIM76H4GrMcYLKhJKtTAlAmBV9pkL82gD4gKpzAEh1IC65j6rKSH54 HcqYUz3ztIV7MEmJ0yA+kl8XY+IFp2RZZh1Chw0f3fRRCACqK00kHJ6NLZ87hQMHNgKfhAyMlj2d MwmkF1uT68LrhUstYoZP1AFQMlvxWqR6wHKAvC54sA+IxGln2DkI5SnK8X0oHl8+QNoe29yY7IC/ bAAb8Ms84Mqx8/8YhNENOCPbX9tLD+b6F/gPqFiQfS/JeQIAAAAASUVORK5CYII= In-Reply-To: (Felix E. Klee's message of "Fri, 12 Jul 2019 16:35:54 +0200") X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 209.51.188.43 X-BeenThere: bug-gnu-emacs@gnu.org List-Id: "Bug reports for GNU Emacs, the Swiss army knife of text editors" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Original-Sender: "bug-gnu-emacs" Xref: news.gmane.org gmane.emacs.bugs:162790 Archived-At: "Felix E. Klee" writes: > If you want to play with the new functions, simply run the code examples > from the info page inside of `example.el`. Concerning `svg+-path`, I=E2= =80=99d > like to hear your thoughts: Do you see room for further improvement, or > could this be the final interface? I haven't read it closely, but it looks good to me. Could you post it as a patch to svg.el? That'll make it easier to read. > I was thinking about removing the double parentheses in commands such > as: > > (lineto ((100 . 200))) > > One could simplify that to: > > (lineto (100 . 200)) > > Polylines would change from: > > (lineto ((100 . 200) (300 . 200))) > > To: > > (lineto (100 . 200) (300 . 200)) > > Reasons against this change: > > * Parsing optional arguments (plist) could be a pain. Currently what > works nicely: > > (lineto ((100 . 200) (200 . 0)) :relative t) Yeah, I think you should keep the argument as a single argument. Varargs with optional keywords parameters just isn't very nice. --=20 (domestic pets only, the antidote for overdose, milk.) bloggy blog: http://lars.ingebrigtsen.no