Skip to content

Commit

Permalink
Remove deprecated -check-printf-calls dflag for LDC
Browse files Browse the repository at this point in the history
It's deprecated since LDC v1.31, as `pragma(printf)` (used by
core.stdc.stdio) is much better and supported by all compilers.
Additionally, I haven't found a single printf occurrence in this
repo here.
  • Loading branch information
kinke authored and veelo committed Mar 31, 2023
1 parent d5d27f0 commit 31e02e2
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 2 deletions.
1 change: 0 additions & 1 deletion dub.json
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@
"importPaths": ["."],
"dflags": ["-ignore"],
"dflags-gdc": ["-ggdb"],
"dflags-ldc": ["-check-printf-calls"],
"configurations": [
{
"name": "default"
Expand Down
1 change: 0 additions & 1 deletion wscript
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,6 @@ def configure(conf):
add_option('-wi')
add_option('-ignore')
add_option('-property')
add_option('-check-printf-calls')
add_option('-g')

if conf.options.mode == 'debug':
Expand Down

3 comments on commit 31e02e2

@nordlow
Copy link
Contributor

@nordlow nordlow commented on 31e02e2 Mar 31, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you be so kind to make a new minor version release and bump code.dlang.org?

@veelo
Copy link
Collaborator

@veelo veelo commented on 31e02e2 Mar 31, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

New release is tagged. I don't own this package so I cannot nudge code.dlang.org, but it should update on its own eventually.

@nordlow
Copy link
Contributor

@nordlow nordlow commented on 31e02e2 Mar 31, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome! Thanks. No tag sync on code.dlang.org yet, though.

Please sign in to comment.