Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

#define snprintf sprintf_s in config.h with Visual Studio not compiling #136

Closed
pelayomendez opened this issue Mar 16, 2016 · 0 comments
Closed

#define snprintf sprintf_s in config.h with Visual Studio not compiling #136

pelayomendez opened this issue Mar 16, 2016 · 0 comments
Labels
Component: build Related to the Makefiles / build scripts in the project. Type: Bug
Milestone
1.4.9

Comments

@pelayomendez
Copy link

While trying to produce a version of my app that uses libmosquitto in windows 10 / Visual studio 2015 I came to this issue regarding "config.h". I had to coment that line:

// #define snprintf sprintf_s

I'm not an expert on the windows side but after googleing a bit I found there are some kind of issue with that define as snprintf implementation is afected by VS version.

http:https://stackoverflow.com/questions/2915672/snprintf-and-visual-studio-2010
@ralight ralight added this to the 1.4.9 milestone Mar 17, 2016
@ralight ralight added Type: Bug Component: build Related to the Makefiles / build scripts in the project. labels Mar 17, 2016
ralight added a commit that referenced this issue Mar 17, 2016
@ralight
Fix building on Visual Studio 2015
6a9382d 
Closes #136.

Thanks to Pelayo Méndez.
@ralight ralight closed this as completed May 14, 2016
@lock lock bot locked as resolved and limited conversation to collaborators Aug 8, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
Component: build Related to the Makefiles / build scripts in the project. Type: Bug
Projects
None yet
Milestone
1.4.9
Development

No branches or pull requests
2 participants
@ralight @pelayomendez