Jean-Marc Lasgouttes <[EMAIL PROTECTED]> writes:
> OK, I read the documentation and came up with the following patch,
> which is probably partly bogus. I think this is supposed to do what we
> need.

Just a reminder for when you come to commit this. The patch as it stands will
produce a couple of compiler warnings under linux because the function arguments
aren't used. You'll need to add a couple of casts to void. (In fact, I think
that there's a BOOST_ARGUMENT_UNUSED macro or somesuch that documents this
kludge quite nicely.

Note also that the comment will need updating ;-)

Angus

+bool canAutoOpenFile(string const & ext, auto_open_mode const mode)
 {
+#ifdef __APPLE__
+ ...
+
+#else
        // currently, no default viewer is tried for non-windows system
        // support for KDE/Gnome/Macintosh may be added later
+       (void)ext;
+       (void)mode;
        return false;
+#endif
 }


Reply via email to