Menu

#365 ufraw.h is not correct for C++

closed-fixed
nobody
None
5
2014-01-12
2014-01-11
No

ufraw fails to build with clang-3.4 due to additional diagnostics. It indicated an error in ufraw.h which is fixed by proper namespacing:

--- ufraw.h.orig 2014-01-11 11:04:08.000000000 -0800
+++ ufraw.h 2014-01-11 11:04:54.000000000 -0800
@@ -41,6 +41,10 @@
/ An impossible value for conf float values /
#define NULLF -10000.0

+#ifdef cplusplus
+extern "C" {
+#endif //
cplusplus
+
/ Options, like auto-adjust buttons can be in 3 states. Enabled and disabled
* are obvious. Apply means that the option was selected and some function
* has to act accourdingly, before changing to one of the first two states
/
@@ -78,10 +82,6 @@ extern UFName ufRawImage;
extern UFName ufRawResources;
extern UFName ufCommandLine;

-#ifdef cplusplus
-extern "C" {
-#endif //
cplusplus
-
UFObject ufraw_image_new();
#ifdef HAVE_LENSFUN
UFObject
ufraw_lensfun_new();

Discussion

  • Jeremy Huddleston Sequoia

    Added patch

     
  • Niels Kristian Bech Jensen

    Thanks for the report and the patch. I have fixed this in the cvs repository.

    Regards,
    Niels Kristian

     
  • Niels Kristian Bech Jensen

    • Status: open --> closed-fixed
     

Log in to post a comment.