[Webkit-unassigned] [Bug 84130] [SOUP] Add a way to register custom uri schemes in WebKit2
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Fri Apr 20 01:02:38 PDT 2012
https://bugs.webkit.org/show_bug.cgi?id=84130
--- Comment #15 from Carlos Garcia Campos <cgarcia at igalia.com> 2012-04-20 01:02:37 PST ---
(In reply to comment #12)
> > > Source/WebKit2/UIProcess/API/C/WKAPICast.h:366
> > > #if defined(BUILDING_GTK__)
> > > #include "WKAPICastGtk.h"
> > > #endif
> > > +
> > > +#if defined(BUILDING_SOUP__)
> > > +#include "WKAPICastSoup.h"
> > > +#endif
> > > +
> >
> > In these cases we should be probably be using PLATFORM(SOUP/GTK) or WTF_PLATFORM_SOUP/GTK instead of duplicating the meaning of that macro with BUILDING_SOUP__.
>
> This is a public header, that's why it uses BUILDING_GTK__ also instead of PLATFORM(GTK)
Ok, switched to use USE(SOUP) in WkAPICast.h, but BUILDING_SOUP is still needed for WKBase.h
> > > Source/WebKit2/WebProcess/soup/WebSoupRequestManager.cpp:74
> > > + genericRequestClass->schemes = const_cast<const char**>(reinterpret_cast<char**>(m_schemes->pdata));
> >
> > The interior cast can be a static_cast here instead of a reinterpret_cast, I believe.
>
> I think I tried that, but I'll try again to be sure.
error: invalid static_cast from type 'void**' to type 'char**'
Only Chuck Norris knows which C++ style cast to use without trying them all until one works :-D
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list