Struts 1.1 To-Do - Enhance role checking

[BONAIUTO,JAMES (HP-NewJersey,ex1)]

I would like to extend the <logic:present> and <logic:notPresent> tags to
accept a comma-delimited list of roles in the role attribute, like this:
                <logic:present 

Is this an acceptable solution to the to-do list item? If so, I'd like to
make that change.

James Bonaiuto