Skip to content

[MINOR] Fix tmpnam -> mkstemp warning#169

Merged
atilaneves merged 1 commit intoatilaneves:masterfrom
dkgroot-dlang:tmpnam2mkstemp
May 13, 2019
Merged

[MINOR] Fix tmpnam -> mkstemp warning#169
atilaneves merged 1 commit intoatilaneves:masterfrom
dkgroot-dlang:tmpnam2mkstemp

Conversation

@dkgroot
Copy link
Contributor

@dkgroot dkgroot commented May 11, 2019

Compiling on Linux Gives

/usr/bin/ld: .dub/obj/d++.o: in function `_D3dpp7runtime3app__T15translationTextTS3std5stdio4FileZQBjFxSQCiQCh7options7OptionsxAyaZ9__lambda8MFNbNiNeZAya':
/data/development/sccp/sources/dlang/dpp/source/dpp/runtime/app.d:160: warning: the use of `tmpnam' is dangerous, better use `mkstemp'

Added a simple replacement to use mkstemp instead (not sure if it is to your liking).
Would have been nice if std.file just had createtmpfile phobos:PR5788 :-)

@dkgroot dkgroot changed the title Fix tmpnam -> mkstemp warning [MINOR] Fix tmpnam -> mkstemp warning May 11, 2019
@atilaneves atilaneves merged commit 5fa6aa4 into atilaneves:master May 13, 2019
@dkgroot
Copy link
Contributor Author

dkgroot commented May 13, 2019

Thanks a lot :-)

@dkgroot dkgroot deleted the tmpnam2mkstemp branch May 13, 2019 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants