HP-41 and 42s: AIP with or without sign?

[Dieter]

(06-07-2018 02:32 PM)Ángel Martin Wrote:  

(06-07-2018 05:50 AM)Valentin Albillo Wrote:  Assuming it doesn't break anything I'd suggest you correct AIP in the SandMatrix to include the sign, as it should and as expected.

A real walk in the park, stay tuned.

Hm. If you really want to change the current Sandmatrix AIP function I think you better rename it (or complement it with a modified function), e.g. AIPS (append integer part with sign). This avoids confusions with the Advantage Pac's AIP which does not preserve the sign.

I wonder if it's a good idea to have different functions with the same name in different ROM modules. What happens if both Sandmatrix and the Advantage Pac are installed and the user calls "AIP"? At the moment at least the results are the same (no sign). If now Sandmatrix gets an AIP with sign support while there also is an AIP without, I can imagine quite a few problems... ;-)

So I think AIP should stay as it is and a second version which honors the sign should be added. Maybe you call it AIPS, or why not simply copy AINT from Sandmath?

Dieter