Comments (4)
I'll close it when I make the error message nicer. Right now the compiler simply throws an exception and immediately exits. It is ugly. I want it to report the error, point to the correct line and continue compiling to search for more errors.
from millfork.
Yes, this is the expected behaviour.
The type of the left side doesn't matter. The type check goes as follows:
246
fits in a byte, therefore it's a byte18
fits in a byte, therefore it's a byte+
has two bytes on its sides, therefore the result of246+18
is a byte- the constant evaluator sees the overflow and informs you that something is wrong
You can get the same issue in C/C++, but the compiler won't warn you. For example, most 32- and 64-bit compilers will compile the following without any warning:
long long sum = 0x8000'0000u + 0x8000'0000u;
to long long sum = 0;
, even thought 0x1'0000'0000u
would fit in a long long
just fine.
The difference is that Millfork does this at every integer size (there are no privileged "default" integer types like int
in C) and doesn't let obvious overflows compile.
from millfork.
Ok! thanks. So you can close this issue.
from millfork.
Error reporting has been improved as of 0.3.12, closing.
from millfork.
Related Issues (20)
- Structs can't have volitile members HOT 1
- Implement ++ and -- operators in the Millfork language HOT 4
- file array initiliser can't handle maths
- Address of sid_v1_sr in file c64_sid.mfk is incorrect HOT 1
- 65CE02 JSR incorrect Assmbly HOT 1
- Multiline comments HOT 5
- possible to pass a function to a macro? or find the identifier for the current function? HOT 3
- Maths doesn't promote to destination type HOT 6
- compiler accepts 'array(type)' but not 'array (type)'
- switch_hirom in c64_easyflash.mfk should write to $DE00 not $DE01
- clear_carry return doesn't work ?
- for x : array strange behavior
- Unnamed Structure and Union Fields
- Other label format support? HOT 3
- Support identity table usage HOT 2
- Assigning struct word member from another struct word member HOT 1
- Compiler assuming C clear when it isn't?
- Hello World example from documentation doesn't work HOT 1
- IX index is broken in the Z80 assembler.
- Strange invalid 16-bit optimization HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from millfork.