Re: [Vimprobable-users] [PATCH RFC] Add command to load per-site cookie files

[Serge E. H. <se...@ha...>]

On Mon, Dec 07, 2015 at 07:12:13PM +0100, Hannes Schüller wrote:
> Hi Serge,
> 
> thanks for this patch!
> 
> "Serge E. Hallyn" <se...@ha...>:
> > The default cookie site is ~/.config/vimprobable/cookies.  With this
> > patch, you can do
> > 
> > :cookies gh
> > or
> > :cookies lp
> > 
> > to load a per-site cookiefile called ~/.config/vimprobable/cookies_gh
> > or cookies_lp (i.e. for github and launchpad).  This allows you to
> > segregate cookies by sites, and more easily keep an eye on suspect
> > sites.
> 
> I basically have two comments.
> 
> 1. I see no way of switching back to the original file.

true.

> 2. What's the rationale for defining this as a new command rather than
>    a "setting"? From your original description, I would have expected
>    something like
>    :set cookiefile=...
>    Don't you think it could be a little confusing to have both
>    :set cookies=(on|off)
>    *and*
>    :cookies=...

In fact it is a little confusing :)  But I did it for lazyness'
sake.  I'll probably have 4-5 cookie files (too many to assign 
ctrl-<something> to each one), and I see no way to map, say,
f6 to ":set cookie_file=" leaving me to fill in the name.  So
':c<tab>' works nicely.

I suppose providing a way (if I'm not wrong, and such a way doesn't
already exist :) to allow such mapping, and making cookiefile a
setting, would make the most sense.