Re: master 6e796b5: Stop project-root from going into infinite recursion

[Stefan Monnier <monnier@iro.umontreal.ca>]

>> I think `cl-call-next-method` is a better choice.
> If you're sure about that.
> The idea here was, some prospective backend author might look at the error
> symbol and make a decision on how to fix the problem based on it.
> And "you need to implement this method" seems better expressed with "No
>  applicable method" rather than with "No next method".

I see.  I consider the difference between the two as an error in CLOS,
but I didn't have the courage to remove it in cl-generic.el ;-)

FWIW, you could also get what you want with something like:

    (cl-defgeneric project-root (project)
      "Return root directory of the current project.
    
    It usually contains the main build file, dependencies
    configuration file, etc. Though neither is mandatory.
    
    The directory name must be absolute.")

    (cl-defmethod project-root (project
                                &context (project--within-roots-fallback
                                          (eql nil)))
      (car (project-roots project)))

This way there indeed won't be any applicable method when
project--within-roots-fallback is non-nil ;-)


        Stefan