> On March 3, 2016, 9:35 p.m., Kevin Funk wrote: > > Ship It!
Oh *facepalm* while pushing I noticed that I had not updated to latest changes and Thomas already fixed this issue,.. Anyhow Thomas patch did not use Q_DECL_IMPORT so I'm pushing that part of this patch. - Andre ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127265/#review93140 ----------------------------------------------------------- On March 3, 2016, 5:49 p.m., Andre Heinecke wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/127265/ > ----------------------------------------------------------- > > (Updated March 3, 2016, 5:49 p.m.) > > > Review request for KDE Frameworks, Andreas Cord-Landwehr and Kåre Särs. > > > Repository: ki18n > > > Description > ------- > > In review Request 127048 (Restrict _nl_msg_cat_cntr use to GNU gettext > implentations.) Unrelated Windows changes were requested and then introduced > which in my opinion were wrong (and in the opinion of my compiler ;-) ). > > You can't just do an extern "C" dllimport declaration inside a function body: > > src/kcatalog.cpp:190:16: error: expected unqualified-id before string constant > extern "C" int __declspec(dllimport) _nl_msg_cat_cntr; > > > Also that patch changed the Logic from an MSVC specific define to general > Q_OS_WIN. This appears to be wrong, too. While I'm only testing with MinGW > I'm pretty sure that the ifndef _MSC_VER was there for a reason. > > This patch reverts the declaration move and logic change while keeping the > __USE_GNU_GETTEXT guard. > > > Diffs > ----- > > src/kcatalog.cpp 083443d > > Diff: https://git.reviewboard.kde.org/r/127265/diff/ > > > Testing > ------- > > Compiled with mingw and __USE_GNU_GETTEXT for Windows. Also compiled for > GNU/Linux. > > > Thanks, > > Andre Heinecke > >
_______________________________________________ Kde-frameworks-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
