Re: [Emacs-diffs] master 3f06795: Migrate auth-source to cl-lib

[Mark Oteiza]

On 12/11/16 at 05:57pm, Stefan Monnier wrote:
> > @@ -712,7 +712,7 @@ must call it to obtain the actual value."
> >                         (plist-get spec key)
> >                         (slot-value backend key))
> >                  (setq filtered-backends (delq backend filtered-backends))
> > -                (return))
> > +                (cl-return))
> >              (invalid-slot-name nil))))
> 
> Hmm... This change looks right, but I believe there's a bug here:
> with `cl`, `dolist` is advised to be CL's `dolist` which includes an
> implicit (block nil ...) wrapper.  But with cl-lib, `dolist` is left alone
> and you need to use `cl-dolist` to get that wrapper.
> 
> So I think you need to change the surrounding `dolist` to be `cl-dolist`
> otherwise this `cl-return` will exit from the surrounding function
> rather than from the inner loop.

Argh, I caught one of the dolist/return but missed this one.
Thanks, fixed.