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