Re: Flymake refactored

[João Távora]

Lele Gaifax <address@hidden> writes:
>
>   (defcustom python-flymake-msg-alist
>     '(("\\(^redefinition\\|.*unused.*\\|used$\\)" . :warning))
>     "Alist used to associate messages to their types.
>   Each element should be a cons-cell (REGEXP . TYPE), where TYPE must be
>   one defined in the variable `flymake-diagnostic-types-alist'."
>     :group 'python-flymake
>     :type '(alist :key-type (regexp)
>                   :value-type (choice (const :tag "Error" :error)
>                                       (const :tag "Warning" :warning)
>                                       (const :tag "Note" :note))))

It's more "should be one defined in the variable". If it's not, it's
treated like an error.

Also, it doesn't have to one of the three default types: python-mode can
very well add some new diagnostic type, :python-silly-note or
:super-fatal-error, that extend the built-in ones.

Just a minor comment, in case you weren't seeing this possibility, and
because the :value-type (choice...) apparently excludes it.

João