Bernhard Roider wrote:
>> In general enums are more explicative:
>>
>> enum CursorMove {
>>     NoMove,
>>     MoveRight,
>>     MoveLeft
>> }
>>
> I did not take a closer look at the code, but shouldn't it be forward
> and backward instead of right and left?
I don't think so. The `right' and `left' don't actually move right or
left but just updates the internal counter pos_. That said, it might be
clearer if these were named posForward() and posBackward(). I can make
that simple change if people wish.

Richard

-- 
==================================================================
Richard G Heck, Jr
Professor of Philosophy
Brown University
http://frege.brown.edu/heck/
==================================================================
Get my public key from http://sks.keyserver.penguin.de
Hash: 0x1DE91F1E66FFBDEC
Learn how to sign your email using Thunderbird and GnuPG at:
http://dudu.dyn.2-h.org/nist/gpg-enigmail-howto

Reply via email to