[Libreoffice] [PATCH] fix cosv build
Tom Tromey
tromey at redhat.com
Wed Aug 10 09:54:20 PDT 2011
I tried building LibreOffice with svn trunk gcc; though in this case the
bug seems to be a fairly generic include order problem.
The build dies in cosv with:
Compiling: cosv/source/strings/string.cxx
In file included from ../../inc/cosv/string.hxx:33:0,
from ../../inc/cosv/csv_precomp.h:39,
from ../inc/precomp.h:32,
from /home/tromey/Space/LibreOffice/bootstrap/clone/sdk/cosv/source/strings/string.cxx:29:
../../inc/cosv/stringdata.hxx: In instantiation of 'csv::StringData<CHAR>::StringData(const CHAR*, csv::StringData<CHAR>::size_type) [with CHAR = char, csv::StringData<CHAR>::size_type = long unsigned int]':
/home/tromey/Space/LibreOffice/bootstrap/clone/sdk/cosv/source/strings/string.cxx:75:17: required from here
../../inc/cosv/stringdata.hxx:105:5: error: 'memcpy' was not declared in this scope, and no declarations were found by argument-dependent lookup at the point of instantiation [-fpermissive]
/usr/include/string.h:44:14: note: 'void* memcpy(void*, const void*, size_t)' declared here, later in the translation unit
dmake: Error code 1, while making '../../unxlngx6.pro/obj/string.obj'
string.hxx includes <string.h>, which declares memcpy, after
cosv/stringdata.hxx. Moving <string.h> earlier fixes the problem.
This is contributed under the LGPLv3+/MPL.
Tom
diff --git a/cosv/inc/cosv/string.hxx b/cosv/inc/cosv/string.hxx
index 08b1220..8f249f9 100644
--- a/cosv/inc/cosv/string.hxx
+++ b/cosv/inc/cosv/string.hxx
@@ -30,9 +30,9 @@
#define COSV_STRING_HXX
// USED SERVICES
+#include <string.h>
#include <cosv/stringdata.hxx>
#include <cosv/str_types.hxx>
-#include <string.h>
#include <cosv/csv_ostream.hxx>
#include <vector>
More information about the LibreOffice
mailing list