Opened 11 years ago

Closed 9 years ago

Last modified 8 years ago

#657 closed task (fixed)

add WARNING pragma

Reported by: ijones Owned by: igloo
Priority: normal Milestone: 6.10.1
Component: Compiler Version: 6.4.1
Keywords: Cc: Bulat.Ziganshin@…
Operating System: Unknown/Multiple Architecture: Unknown/Multiple
Type of failure: None/Unknown Test Case: N/A
Blocked By: Blocking:
Related Tickets: Differential Rev(s):
Wiki Page:

Description

It might be nice to have a WARNING pragma in place of or in addition to the DEPRECATED pragma that just allows you to issue a generic warning:

{-# WARNING f, C, T "These are deprecated" #-}
{-# WARNING g "This has terrible performance" #-}
{-# WARNING T "We had to expose this type for bad reasons, don't use it." #-}
{-# WARNING preConf "This is an unstable interface. May become deprecated." #-}
{-# WARNING fromJust "Calls the error function.  use maybe for safety" #-}
module Wibble {-# WARNING "Use Wibble_1 instead" #-}

This is more general than the DEPRECATED pragma.

Change History (10)

comment:1 Changed 11 years ago by simonpj

This could be easily done if we can agree about what we want.

Should "WARNING" be "WARN_USE" or "WARN_USE_OF" or "WARN_AT_USE" or something like that? Something more self explanatory.

(Or use hyphens? "WARN-USE" or "WARN-USE-OF" etc.)

Then DEPRECATED becomes "WARN-USE" with an automatic prefix "x is deprecated.". Right?

comment:2 Changed 11 years ago by igloo

Milestone: 6.6.1
Test Case: N/A

comment:3 Changed 11 years ago by simonmar

Milestone: 6.6.16.8

comment:4 Changed 10 years ago by guest

Cc: Bulat.Ziganshin@… added

comment:5 Changed 10 years ago by simonmar

Milestone: 6.8 branch6.10 branch
Owner: set to igloo

comment:6 Changed 9 years ago by igloo

Milestone: 6.10 branch6.10.1

comment:7 Changed 9 years ago by igloo

Resolution: fixed
Status: newclosed

WARNING pragma added

comment:8 Changed 9 years ago by simonmar

Architecture: UnknownUnknown/Multiple

comment:9 Changed 9 years ago by simonmar

Operating System: UnknownUnknown/Multiple

comment:10 Changed 8 years ago by simonmar

difficulty: Easy (1 hr)Easy (less than 1 hour)
Note: See TracTickets for help on using tickets.