From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Eli Zaretskii Newsgroups: gmane.emacs.devel Subject: Re: [PATCH] add emacsclient support to open with file:linum syntax Date: Fri, 08 Jan 2016 21:43:04 +0200 Message-ID: <83ziwfoo1j.fsf@gnu.org> References: <838u40qw8j.fsf@gnu.org> <837fjkozax.fsf@gnu.org> <56900BBE.8080804@porkrind.org> Reply-To: Eli Zaretskii NNTP-Posting-Host: plane.gmane.org X-Trace: ger.gmane.org 1452282200 11853 80.91.229.3 (8 Jan 2016 19:43:20 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Fri, 8 Jan 2016 19:43:20 +0000 (UTC) Cc: jorge.garcia.gonzalez@gmail.com, yuri.v.khan@gmail.com, emacs-devel@gnu.org, kaushal.modi@gmail.com To: David Caldwell Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Fri Jan 08 20:43:15 2016 Return-path: Envelope-to: ged-emacs-devel@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1aHcwM-0001wX-KO for ged-emacs-devel@m.gmane.org; Fri, 08 Jan 2016 20:43:14 +0100 Original-Received: from localhost ([::1]:37858 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aHcwM-00088k-0s for ged-emacs-devel@m.gmane.org; Fri, 08 Jan 2016 14:43:14 -0500 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:55700) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aHcwI-00088T-6Q for emacs-devel@gnu.org; Fri, 08 Jan 2016 14:43:10 -0500 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1aHcwF-0006zv-0d for emacs-devel@gnu.org; Fri, 08 Jan 2016 14:43:10 -0500 Original-Received: from fencepost.gnu.org ([2001:4830:134:3::e]:34988) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aHcwE-0006zr-Td; Fri, 08 Jan 2016 14:43:06 -0500 Original-Received: from 84.94.185.246.cable.012.net.il ([84.94.185.246]:1659 helo=HOME-C4E4A596F7) by fencepost.gnu.org with esmtpsa (TLS1.2:RSA_AES_128_CBC_SHA1:128) (Exim 4.82) (envelope-from ) id 1aHcwE-0007DM-1C; Fri, 08 Jan 2016 14:43:06 -0500 In-reply-to: <56900BBE.8080804@porkrind.org> (message from David Caldwell on Fri, 8 Jan 2016 11:19:26 -0800) X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 2001:4830:134:3::e X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Original-Sender: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.devel:197848 Archived-At: > Cc: Eli Zaretskii , Yuri Khan , > Emacs developers > From: David Caldwell > Date: Fri, 8 Jan 2016 11:19:26 -0800 > > Why not instead of adding the new option, rearrange the logic so that it > only tries to parse the ":linenum" part if the file doesn't exist? Because both 'filename' and 'filename:1234' could exist. Really, I don't understand why should we come with fancy syntax when a new option will unequivocally tell which case is it. Learning a new option is not harder than learning a new syntax that wasn't supported before.