Bug 117360 - Missing extern "C" for hspell include
Summary: Missing extern "C" for hspell include
Status: RESOLVED FIXED
Alias: None
Product: kdelibs
Classification: Unmaintained
Component: kspell (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Zack Rusin
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-11-30 17:20 UTC by Adam Sampson
Modified: 2009-02-12 23:08 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Adam Sampson 2005-11-30 17:20:37 UTC 
Version:            (using KDE KDE 3.5.0)
Installed from:    Compiled From Sources
Compiler:          gcc (GCC) 3.4.4 
OS:                Linux

In kspell_hspellclient.h, there's:

#include "hspell.h"

Since hspell is a C library, that causes the hspell plugin to fail to link with a load of unresolved symbol errors. It needs wrapping with extern "C", as kspell_hspelldict.h already does:

extern "C" {
#include "hspell.h"
}
Comment 1 Thiago Macieira 2005-12-06 17:07:17 UTC 
SVN commit 486041 by thiago:

Fixing the bug in kspell's hspell plugin. Thanks to Jeff Manson for the patch.

BUG:117329,117722,117360


 M  +3 -0      kspell_hspellclient.h  


--- trunk/KDE/kdelibs/kspell2/plugins/hspell/kspell_hspellclient.h #486040:486041
@@ -25,7 +25,10 @@
 #include "client.h"
 #include <qobject.h>
 
+/* libhspell is a C library and it does not have #ifdef __cplusplus */
+extern "C" {
 #include "hspell.h"
+}
 
 namespace KSpell2 {
     class Dictionary;
Comment 2 John Layt 2009-02-12 23:08:59 UTC 
*** Bug 177000 has been marked as a duplicate of this bug. ***