CompilerWarning

[kenmo]

PureBasic already has CompilerError, which allows us to force an error if the person compiling should be alerted of something. It would be nice if we could force a CompilerWarning too, which would not prevent compilation, but it would still present an obvious warning to the user that something is wrong or important.

It would be similar to the warnings that appear when a resource fails to load:

Code: Select all

If LoadImage(0, "doesNotExist.bmp")
  Debug "A warning should appear in the IDE."
EndIf

Also, this is a very minor punctuation request, but I wish the period "." was not automatically added to our CompilerError strings, such as:

Code: Select all

CompilerError "Fatal Error!"

[Shield]

+1, good points.

[ts-soft]

WorkAround (crossplattform)

Code: Select all

CompilerIf #PB_Compiler_Debugger
  CompilerSelect #PB_Compiler_OS
    CompilerCase #PB_OS_Windows
      Import ""
         PB_DEBUGGER_SendWarning(Message.p-ascii)
       EndImport
    CompilerDefault
      ImportC ""
         PB_DEBUGGER_SendWarning(Message.p-ascii)
       EndImport
   CompilerEndSelect
CompilerEndIf

Macro CompilerWarning(error)
  CompilerIf #PB_Compiler_Debugger
    PB_DEBUGGER_SendWarning(error)
  CompilerEndIf
EndMacro

CompilerWarning("A warning should appear in the IDE.")
   

[kenmo]

That's a useful tip, thank you.

I always see people posting these Import "" tips and I don't know how they know about them! Are they documented somewhere?

[ts-soft]

To import a function from a lib, that is imported, no name for the lib required.

Code: Select all

CompilerIf #PB_Compiler_Debugger

make sure, the lib is imported!

[wilbert]

WorkAround (crossplattform)

It's not entirely cross platform.
For OS X you need to use ImportC .

[ts-soft]

It's not entirely cross platform.
For OS X you need to use ImportC .

Thx, changed
(i can't make any test on macos)

[wilbert]

Thx, changed

It's working fine now on OS X

[Little John]

Done.

I just found out that this has been added in PB 5.40 LTS (tested with the Windows versions).

Code: Select all

CompilerWarning "Hi there"

I can't see where it is documented, though.

[StarBootics]

Done.

I just found out that this has been added in PB 5.40 LTS (tested with the Windows versions).

Also work on Linux (Ubuntu Gnome).
Just nice !

Best regards
StarBootics

[freak]

Please note that what was implemented here is a compiler warning. This means the warning is triggered at compile-time and you have to use things like CompilerIf if you want to show it conditionally. What ts-soft posted is code to trigger a debugger warning. This warning is issued by the debugger at run-time, and it will only show if the statement is actually executed (so you can show it conditionally using a regular if). There seems to be some confusion between the two at times.

For completeness, i just added two functions DebuggerError(Message$) and DebuggerWarning(Message$) to trigger debugger errors (in essence what ts-soft showed as an official function). This will be in v5.50.

[NicTheQuick]

Nice!