I have declared an attribute as such:

CLASS_ATTR_LONG(c,      	"myattr", 	0, t_myobj, myattr);
CLASS_ATTR_ACCESSORS(c,		"myattr",    	(method)myobj_myattr_get, (method)myobj_myattr_set);
CLASS_ATTR_ENUMINDEX(c,		"myattr",	0, "Left Center Right");
CLASS_ATTR_DEFAULT_SAVE_PAINT(c,"myattr",    	0, "0");

The custom getter and setter:

t_max_err myobj_myattr_get(t_myobj *x, t_object *attr, long *argc, t_atom **argv)
{
    char alloc;

    if (argc && argv) {
	if (atom_alloc(argc, argv, &alloc)) {
            return MAX_ERR_OUT_OF_MEM;
        }
        if (alloc) {
            atom_setlong(*argv, x->myattr);
	}
    }
    return MAX_ERR_NONE;
}

t_max_err myobj_myattr_set(t_myobj *x, t_object *attr, long argc, t_atom *argv)
{
    if (argc && argv) {
        x->myattr = atom_getlong(argv);
    }
    return MAX_ERR_NONE;
}

Pretty straightforward, however I have one problem:
if I understand everything clearly, the CLASS_ATTR_ENUMINDEX macro creates an “enumindex” attribute, which becomes an attribute of the “myattr” attribute.
However it seems I am having problems updating the “enumindex” attribute when I use a custom getter and setter.

In other words, when the user clicks on an inlet of my object and the quickref menu gets displayed, the string portion of “myattr” always says “Left” even though the numeric value associated with “myattr” might change. I also see a ‘bad number’ error in the Max window.

So I am guessing that there must be a way to update the “enumindex” attribute after changing the value of myattr in order for the two to be synced together.
Probably I need to do it manually since I defeat the internal setter and getter. But how ???

Any suggestions ?

Thanks.

- Luigi

CLASS_ATTR_ACCESSORS(c,		"myattr",    	(method)NULL, (method)myobj_myattr_set);

In this case it uses the standard gettr method.

- Luigi

i did something like that ; maybe have a look in the sources of my [november] external ;
sorry don’t have time to make a cleaner example ;-)

HTH

EDIT : oops / not ENUMINDEX in my code but ENUM …

t_max_err myobj_myattr_get(t_myobj *x, t_object *attr, long *argc, t_atom **argv)
{
    char alloc;

    if (argc && argv) {
	if (atom_alloc(argc, argv, &alloc)) {
            return MAX_ERR_GENERIC;
        }
        atom_setlong(*argv, x->myattr);
    }
    return MAX_ERR_NONE;
}

However the above custom getter produces an “enumindex: bad number” error in the Max window and fails to update the symbol portion of the attribute.

Curiously enough, if the same attribute is declared as ENUM (with a symbol datatype) everything works fine.

To better illustrate the problem I wrote a simple UI external with just one attribute defined as ENUMINDEX with custom getter and setter.

If you compile it and then open the help file you will find instructions to reproduce the problem exactly as it appears to me.

Please, let me know what you guys think…

Thanks a lot.

- Luigi

problem comes from if (alloc) statement in you code :

if (alloc) {
atom_setlong(*argv, x->state);
}

as EJ said ;-)

atom_setlong(*argv, x->state);

IMHO, you don’t need to test alloc as if memory have been passed you get zero, but you still need to set the atom passed ; that seems to happen in your method ; so you get bad number error when you query attributes with the inlet popup window …

expert opinion ?

Emmanuel and Vanille, thank you so much for your help.

- Luigi