Comment 6 for bug 742985

Revision history for this message
Marco Biscaro (marcobiscaro2112) wrote :

Yes, I saw. Let me try to explain what I've done:

I removed the != 0 because the condition > 32 includes it.

And, why > 32? ASCII codes: http://bestofthisweb.com/blogs/wp-content/uploads/2009/11/ascii_table2.jpg

Any value less than or equal to 32 is invalid for shortcuts (they're control codes and space). This way, even if a invalid shortcut is set by mistake, the black box won't be rendered. Got it?