Project

General

Profile

Bug #540

Colour command no longer works.

Added by Brian Flanagan over 3 years ago. Updated over 3 years ago.

Status:
Feedback
Priority:
Low
Target version:
Start date:
10/04/2020
Due date:
% Done:

0%

Estimated time:
1:00 h
Affected version:

Description

Screen Open 0,320,200,32,Lowres
Colour(17)=$F


Files

ColourBug2.png (26.2 KB) ColourBug2.png And the generated code that caused it. Brian Flanagan, 10/05/2020 01:26 AM
ColourBug1.png (60.8 KB) ColourBug1.png Here's the error... Brian Flanagan, 10/05/2020 01:26 AM
#1

Updated by Francois Lionet over 3 years ago

  • Status changed from New to Resolved
  • Assignee set to Francois Lionet
  • Target version set to Beta RC4

Fixed

#2

Updated by Francois Lionet over 3 years ago

  • Status changed from Resolved to Feedback
  • Assignee changed from Francois Lionet to Brian Flanagan

ERRATA. I fixed it, then had a look in the AMOSPro manual.

Colour is NOT a reserved variable. You cannot do Colour(XX) = YY . YOu have an instruction and a function.
Colour XX, YY <- the instruction
Print Colour( XX ) <- the function.

Where did you see that it was a reserved variable ?

#3

Updated by Brian Flanagan over 3 years ago

  • Priority changed from High to Low

When I saw your note, I went back and checked the AMOSPro manual as well. My mistake! I usually double-check the syntax before posting a bug. I must have missed this one. I don't know if I read this in another doc or I just remembered wrong. Sorry about that! I hope you didn't work too long on this.

Just a thought though, (especially since you already did it), is there any reason we could NOT do this? Since the palette is already treated like an array with c=Colour(n), it is intuitive that Colour(n)=c should also be valid.
It would be a plus for ease of use. (Of course, we'd keep the original syntax for AMOSPro compatibility.)

What do you think?

#4

Updated by Brian Flanagan over 3 years ago

(Near the end of my last comment "original syntax" should read "instruction syntax").

Also available in: Atom PDF