WP 34S and 31S bugs and fixes

[Bit]

(01-08-2015 04:38 PM)walter b Wrote:  

(01-08-2015 07:09 AM)walter b Wrote:  

(01-08-2015 05:28 AM)Bit Wrote:  

1. Incomplete or invalid fractions cause delayed error messages ...
   

Your suggestion concerning 1. sounds reasonable but I need some time to check. Please be patient here, I think I can do it this evening.

Ok, I also agree on your suggestion for 1.

d:-)

My original patch worked in such a way (because it was simpler to implement) that whenever the command line contained an invalid fraction, a domain error would be produced when you pressed the button for any operation, not when the button was released. Thus the names of operations wouldn't be displayed and they couldn't be canceled (NULL-ed). While that's an improvement, it's not the most elegant solution because it isn't consistent with how the calculator usually behaves.

I created another patch (it needs a bit more testing) that's better IMO: Error messages are only produced when you try to execute an operation by releasing the button, but not if you cancel it. (Keys that immediately do something like entering a catalog still produce an error message on key press.) The cost is roughly 50 bytes in the firmware because the function that verifies the command line needs to be invoked from more places.

Which behavior should be adopted?