Thread Rating:
  • 0 Vote(s) - 0 Average
  • 1
  • 2
  • 3
  • 4
  • 5
FASTCALL warnings
#1
Hi,

Whenever compiling any ZX BASIC program that uses the latest BIFROST* interface library, it will always produce the following warnings:

Code:
bifrost.bas:92: warning: FUNCTION 'BIFROSTfindAttr' declared as FASTCALL with 2 parameters
bifrost.bas:177: warning: SUB 'BIFROSTshowTile' declared as FASTCALL with 3 parameters
bifrost.bas:201: warning: SUB 'BIFROSTshowTilePos' declared as FASTCALL with 2 parameters
bifrost.bas:231: warning: SUB 'BIFROSTfillTileAttr' declared as FASTCALL with 3 parameters
In general, these warnings are very useful to notify the user about possible problems on the code. But in the cases when they were already validated (such as above where FASTCALL is used intentionally for performance reasons), they become a disadvantage: the user will always see warnings in every compilation and will soon start to ignore them completely, thus not paying attention anymore to new warnings that may appear about real problems in his/her own code.

I suggest a new directive to allow ignoring such warnings inside an specific library. The idea is that a library file could use something like this:

Code:
#pragma push(fastcall_warning)
#pragma fastcall_warning = false
...
#pragma pop(fastcall_warning)
Makes sense?
Reply
#2
Definitely! In fact, I'm implementing some enable/disable Warning flags (such as "Unused function", which is very handy for libraries, or this one).
They're on the way! Smile
Reply
#3
boriel Wrote:Definitely! In fact, I'm implementing some enable/disable Warning flags (such as "Unused function", which is very handy for libraries, or this one).
They're on the way! Smile
Thank you!
Reply
#4
I could not find any information in ZX BASIC wiki about warning flags or pragma directives... so I'm wondering if this feature has been implemented already?
Reply
#5
einar Wrote:I could not find any information in ZX BASIC wiki about warning flags or pragma directives... so I'm wondering if this feature has been implemented already?
Sorry, I'm afraid not. :oops:
I stopped any further development on version 1.x.
Also I'm currently on vacation :roll:
Reply
#6
No problem, thanks for the reply.

Enjoy your vacation! Smile
Reply
#7
Einar, hope you're still alive :|
Now that ZX BASIC 1.4 is out, I think we can start adding this feature. :wink:
Reply
#8
boriel Wrote:Now that ZX BASIC 1.4 is out, I think we can start adding this feature. :wink:

Excellent!
Reply


Forum Jump:


Users browsing this thread: 2 Guest(s)