Jean-Marc Lasgouttes wrote:

>>>>>> "Angus" == Angus Leeming <[EMAIL PROTECTED]>
>>>>>> writes:
> 
> Angus> The patch is nice in that it returns a container of
> Angus> kb_sequences when youask for a container of kb_sequences,
> Angus> rather than some stringified version of the same.
> 
> Yes. But I thought we were not in 'make nice code' mode anymore.

This piece of code cannot modify anything. That, after all, is the whole
point. It simply provides a mechanism to iterate over immutable
LyXKeySyms.

> Angus> // This iterator, when de-referenced,
> Angus> // returns 'LyXKeySym const &', which is exactly what
> Angus> // you're interested in. Moreover, the returned data is
> Angus> // immutable, also something that's good here.
> 
> And what about the modifiers? I am interested in them too.

What do you want? An accessor to the 'first' variable in each element of
modifiers? As in
        buf += kb_keymap::printKeySym(*sequence[i], modifiers[i].first);
?

-- 
Angus

Reply via email to