Re: [PATCH] add emacsclient support to open with file:linum syntax

[Jorge Alberto Garcia <jorge.garcia.gonzalez@gmail.com>]

On Tue, Mar 29, 2016 at 5:08 PM, Paul Eggert <eggert@cs.ucla.edu> wrote:
> On 03/29/2016 03:35 PM, Kaushal Modi wrote:
>>
>>
>> I myself, use an awk script to rearrange the arguments to work with the
>> current +LINE FILE format accepted by emacsclient ( Ref:
>> https://lists.gnu.org/archive/html/emacs-devel/2016-01/msg00541.html ).
>
>
> So the existing emacsclient implementation already works for you.
>
>> Having this patch installed will prevent the need for such workarounds
>
>
> They are not workarounds, because grep output cannot be used unchanged. Even
> with the proposed option syntax, users would need to employ a script such as
> yours to pick out the part of the grep output that they need, and use that
> information to formulate arguments for emacsclient.

It can, you just assumed you need to use a script,
If you have this patch you can copy/paste the result paths and use it
directly with emacsclient

without having to form +line:column  and select/copy/paste the path again.

>
> The longstanding +LINE option syntax is compatible with other popular free
> editors such as 'vim' and 'nano', and there is some value in encouraging its
> usage rather than reinventing the wheel.

The patch does not remove the current +LINE:COL syntax support
so it adds compatibility with a new way to specify it.