http://codereview.appspot.com/4638052/diff/1/samplecode/SampleApp.cpp File samplecode/SampleApp.cpp (right): http://codereview.appspot.com/4638052/diff/1/samplecode/SampleApp.cpp#newcode... samplecode/SampleApp.cpp:1060: case 'e': Why 'e'? Something like 'p' would seem to make more sense.
http://codereview.appspot.com/4638052/diff/1/samplecode/SampleApp.cpp File samplecode/SampleApp.cpp (right): http://codereview.appspot.com/4638052/diff/1/samplecode/SampleApp.cpp#newcode... samplecode/SampleApp.cpp:1060: case 'e': On 2011/06/21 14:33:27, djsollen wrote: > Why 'e'? Something like 'p' would seem to make more sense. Agreed. 'p' is used for "Pipe", though. Only so many letters in the alphabet, and this was one of the few remaining. Ideally, we would have a menu, instead of hardcoding letter identifiers, and we do have an action bar menu option on Android
lgtm