Re: [O] [PATCH] * org-insert-link: use ido when inserting links

2012-10-13 Thread Nicolas Goaziou
Hello, tony day writes: > And a revised patch for your reviewing pleasure :) Ok. I have applied it and added you to the list of contributors. Thank you again for the patch. Regards, -- Nicolas Goaziou

[O] [PATCH] * org-insert-link: use ido when inserting links

2012-10-12 Thread tony day
On 12 Oct 2012, at 23:50, Nicolas Goaziou wrote: > Thanks for considering my suggestions. So here are new ones: And a revised patch for your reviewing pleasure :) > Judging by your test case, it's the > > (mapcar 'cadr org-stored-links) > > part that cause problems, isn't it? I'm not sure w

Re: [O] [PATCH] * org-insert-link: use ido when inserting links

2012-10-12 Thread Nicolas Goaziou
Hello, Thanks for considering my suggestions. So here are new ones: tony day writes: > org.el (org-insert-link): removed a list within the list of link > creation that was causing a bug when using ido. Nitpick: "Removed a list...". Judging by your test case, it's the (mapcar 'cadr org-stor

[O] [PATCH] * org-insert-link: use ido when inserting links

2012-10-11 Thread tony day
org.el (org-insert-link): removed a list within the list of link creation that was causing a bug when using ido. Removed the hard coded iswitch and ido switches. Changed the order of prefixes so http came up first. (org-iread-file-name): created a function that can use ido-read-file-name if flag