Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

In compiled Red/System negate of logic value sometimes does not work #1481

Closed
Oldes opened this issue Dec 3, 2015 · 1 comment
Closed

In compiled Red/System negate of logic value sometimes does not work #1481

Oldes opened this issue Dec 3, 2015 · 1 comment
Assignees
@dockimbel
Labels
status.built A change in codebase has been done to address the ticket. status.tested The change in code has been manually tested and verified to fix the issue. type.bug Ticket describes an abnormal behavior, not conforming to the specs or expectation.
Milestone
0.6.0

Comments

@Oldes
Copy link
Contributor

Oldes commented Dec 3, 2015 

Red/System []

i: 1
case?: as logic! (i + 1)
probe i
probe case?
probe not case?

i: 2
case?: as logic! i
probe i
probe case?
probe not case?

i: 3
case?: as logic! i
probe i
probe case?
probe not case?

Outputs:

1
true
true ;<----------- this is wrong!!!
2
true
false
3
true
false
@dockimbel dockimbel added type.bug Ticket describes an abnormal behavior, not conforming to the specs or expectation. status.reviewed Ticket has been reviewed and accepted for further processing. Red labels Jan 15, 2016
@dockimbel dockimbel self-assigned this Jan 15, 2016
@dockimbel dockimbel mentioned this issue Jan 15, 2016
Closed
@qtxie qtxie mentioned this issue Jan 25, 2016
Closed
@dockimbel
Copy link
Member

The issue comes from a missing generation of runtime type casting for in:

case?: as logic! (i + 1)

@dockimbel dockimbel added this to the 0.6.0 milestone Feb 5, 2016
dockimbel added a commit that referenced this issue Feb 5, 2016
@dockimbel
FIX: issue #1481 (In compiled Red/System negate of logic value someti…
587aa76 
…mes does not work)
@dockimbel dockimbel added status.built A change in codebase has been done to address the ticket. status.tested The change in code has been manually tested and verified to fix the issue. and removed status.reviewed Ticket has been reviewed and accepted for further processing. labels Feb 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dockimbel dockimbel

Labels
status.built A change in codebase has been done to address the ticket. status.tested The change in code has been manually tested and verified to fix the issue. type.bug Ticket describes an abnormal behavior, not conforming to the specs or expectation.
Projects
None yet
Milestone
0.6.0
Development

No branches or pull requests
2 participants
@dockimbel @Oldes