[Okular-devel] Review Request 125741: okular: Fix linking with MSVC 2015.

[Albert Astals Cid]

> On oct. 21, 2015, 10:19 p.m., Albert Astals Cid wrote:
> > This is a workaround and will break when i decide to remove it again becuase the include is not needed there, are you sure there isn't a better way to fix it?
> 
> Gleb Popov wrote:
>     Hum. It's possible not to use generated export header, but make this class exported unconditionally:
>     
>     ```
>     #ifdef _MSC_VER
>     #define VIEWERINTERFACE_EXPORT __declspec(dllexport)
>     #else
>     #define VIEWERINTERFACE_EXPORT __attribute__((visibility("default")))
>     #endif
>     
>     class VIEWERINTERFACE_EXPORT ViewerInterface
>     ```

in the else branch i'd make more sense to have the current export, no?

Also please start to fill in the Branch in the review requests you create, it's confusing knowing if sometimes stuff is for the frameworks branch or for master.


- Albert


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125741/#review87233
-----------------------------------------------------------


On oct. 21, 2015, 6:07 p.m., Gleb Popov wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125741/
> -----------------------------------------------------------
> 
> (Updated oct. 21, 2015, 6:07 p.m.)
> 
> 
> Review request for Okular.
> 
> 
> Repository: okular
> 
> 
> Description
> -------
> 
> okular/part.cpp includes interfaces/viewerinterface.h and its OKULARCORE_EXPORT macro gets expanded into __declspec(dllimport). This makes VS linker to drop ViewerInterface::~ViewerInterface() definition and to look for it in KF5OkularCore.dll. But this include is never used anywhere in okularcore, hence missing symbol.
> 
> Including interfaces/viewerinterface.h in any of okularcore sources fixes the problem.
> 
> 
> Diffs
> -----
> 
>   core/document.cpp 49ce0650b4f970290490beb0db6d5c29c13311fa 
> 
> Diff: https://git.reviewboard.kde.org/r/125741/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Gleb Popov
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/okular-devel/attachments/20151022/3cadb592/attachment.html>