Re: Compilation warnings in mouse.el

[Stefan Monnier]

EZ> Unlike Stefan, I think we should add :group to every defcustom. Having
EZ> them mysteriously missing from some of them is a time bomb: 

The way I see it, defcustoms should pretty much never have :group, and
the group to which they belong is simply determined by the file in which
they occur.

E.g. the `mouse' group should be defined in mouse.el rather than in
cus-edit.el, which would have solved this problem.

EZ> remove or add
EZ> enough defcustoms in the same file, and you have a bug.

I don't see how removing/adding defcustoms in the same file
would introduce problems.


        Stefan


PS: Incidentally, we have two definitions of the `mouse' group
in cus-edit.el:

    % grep -A2 '(defgroup mouse ' **/*.el
    lisp/cus-edit.el:(defgroup mouse nil
    lisp/cus-edit.el-  "Mouse support."
    lisp/cus-edit.el-  :group 'editing)
    --
    lisp/cus-edit.el:(defgroup mouse nil
    lisp/cus-edit.el-  "Input from the mouse."
    lisp/cus-edit.el-  :group 'environment)
    %