Bug 53716 - define error in kcmlocale.h
Summary: define error in kcmlocale.h
Status: RESOLVED FIXED
Alias: None
Product: kde
Classification: I don't know
Component: general (show other bugs)
Version: unspecified
Platform: Compiled Sources IRIX
: NOR normal
Target Milestone: ---
Assignee: Unassigned bugs mailing-list
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-01-31 08:02 UTC by Jonah Michaud
Modified: 2003-01-31 11:44 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jonah Michaud 2003-01-31 08:02:51 UTC
Version:            (using KDE KDE 3.1)
Installed from:    Compiled From Sources
Compiler:          gcc 3.2.1 
OS:          Irix

diff kdebase-3.1/kcontrol/locale/kcmlocale.h kdebase-3.1/kcontrol/locale/kcmlocale.h.orig
2c2
<  * kcmlocale.h
---
>  * locale.h
26,27c26,27
< #ifndef __KCMLOCALE_H__
< #define __KCMLOCALE_H__
---
> #ifndef __LOCALE_H__
> #define __LOCALE_H__

The original fails with the following error:

toplevel.cpp: In constructor `KLocaleApplication::KLocaleApplication(QWidget*, 
   const char*, const QStringList&)':
toplevel.cpp:65: invalid use of undefined type `struct KLocaleConfig'
toplevel.h:36: forward declaration of `struct KLocaleConfig'
toplevel.cpp: In member function 
   `KLocaleApplication::KLocaleApplication(QWidget*, const char*, const 
   QStringList&)':
toplevel.cpp:53: warning: unused parameter `const char*name'
toplevel.cpp: In member function `virtual void KLocaleApplication::save()':
toplevel.cpp:183: invalid use of undefined type `struct KLocaleConfig'
toplevel.h:36: forward declaration of `struct KLocaleConfig'

I've encoutered a few other errors compiling kdelibs and kdebase, but perhaps they are irix-specific.  I'll submit those after I finish compiling everything.  The error above seems like it would be cross-platform.

(I'm using gcc-3.2.1 on irix 6.5)
Comment 1 Waldo Bastian 2003-01-31 11:44:54 UTC
Subject: KDE_3_1_BRANCH: kdebase/kcontrol/locale

CVS commit by waba: 

CCMAIL: 53716-done@bugs.kde.org
Thanks for reporting.
-#ifndef __LOCALE_H__
-#define __LOCALE_H__
+#ifndef __KCMLOCALE_H__
+#define __KCMLOCALE_H__


  M +3 -3      kcmlocale.h   1.39.4.1


--- kdebase/kcontrol/locale/kcmlocale.h  #1.39:1.39.4.1
@@ -1,4 +1,4 @@
 /*
- * locale.h
+ * kcmlocale.h
  *
  * Copyright (c) 1998 Matthias Hoelzer <hoelzer@physik.uni-wuerzburg.de>
@@ -24,6 +24,6 @@
 
 
-#ifndef __LOCALE_H__
-#define __LOCALE_H__
+#ifndef __KCMLOCALE_H__
+#define __KCMLOCALE_H__
 
 #include <qwidget.h>