Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

removed unneeded config.h include in AddonString.h. #3358

Merged
merged 1 commit into from

2 participants

wsoltys Trent Nelson
wsoltys
Collaborator

this fixes the build error on windows. I can make it windows only but its probably not needed for the other platforms as well.
Windows seemed to use the config.h from pcre which is no longer and thus the builds fails.

wsoltys
Collaborator

If UNICODE is defined via config.h for other platforms we might guard it with #ifdef HAVE_CONFIG_H

Trent Nelson
wsoltys
Collaborator

I wrapped the config.h in HAVE_CONFIG_H and merge it in a few if nobody objects.

wsoltys wsoltys merged commit 710b1ee into from
Deleted user Unknown referenced this pull request from a commit
Commit has since been removed from the repository and is no longer available.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Oct 2, 2013
This page is out of date. Refresh to see the latest.
Showing with 2 additions and 0 deletions.
  1. +2 −0  xbmc/interfaces/legacy/AddonString.h
2  xbmc/interfaces/legacy/AddonString.h
View
@@ -21,7 +21,9 @@
#pragma once
#include <string>
+#ifdef HAVE_CONFIG_H
#include "config.h"
+#endif
namespace XBMCAddon
{
Something went wrong with that request. Please try again.