bug#28489: Acknowledgement (27.0.50; eieio-persistent slot type validation should be a bit smarter)

[Noam Postavsky]

Eric Abrahamsen <eric@ericabrahamsen.net> writes:

>       ((eq (car-safe type) 'or)
> -      ;; If type is a list, and is an or, it is possibly something
> -      ;; like (or null myclass), so check for that.
> -      (let ((ans nil))
> -        (dolist (subtype (cdr type))
> -          (setq ans (eieio-persistent-slot-type-is-class-p
> -                     subtype)))
> -        ans))
> +      ;; If type is a list, and is an or, it is possible that
> +      ;; multiple classes are acceptable, find them all.
> +      (seq-filter (lambda (elt) (class-p elt)) (cdr type)))

You seem to have removed some recursion here, is that correct?  If yes,
probably something worth explaining in the commit message.

Minor nitpicks:
- The lambda could be replaced with just #'class-p.
- The indentation has a mix of tabs and spaces.