Skip to content

Commit

Permalink
Fixes to stpcpy compat wrapper following review
Browse files Browse the repository at this point in the history
  • Loading branch information
@hoglet67
hoglet67 committed Feb 28, 2017
1 parent d9dfaee commit 9f5daaa
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
4 changes: 2 additions & 2 deletions src/compat_wrappers.h
View file
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,8 @@

#include "b-em.h"

#ifndef WIN32
#define HAVE_STPCPY
#ifdef WIN32
#undef HAVE_STPCPY

This comment has been minimized.

Sign in to view

Copy link
@ThomasAdam

ThomasAdam Feb 28, 2017

Contributor

Is this right -- I suppose you'll have to wait till I've merged @SteveFosdick's other changes in, but this now says that if WIN32 is defined, that you won't need to check for HAVE_STPCPY because Windows has this already. Is this true?
#endif

FILE *x_fopen(const char *, const char *);
Expand Down
1 change: 0 additions & 1 deletion src/vdfs.c
View file
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,6 @@
#include "mem.h"
#include "model.h"
#include "tube.h"
#include "compat_wrappers.h"

#include <ctype.h>
#include <errno.h>
Expand Down

1 comment on commit 9f5daaa

@SteveFosdick
Copy link
Collaborator

Choose a reason for hiding this comment

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

@ThomasAdam: No, it's #undef. If platform is Windows we know we don't have the function.

Please sign in to comment.