Review Request 127187: Fix build for MSVC (2013) on Windows

Thomas Friedrichsmeier thomas.friedrichsmeier at ruhr-uni-bochum.de
Sat Feb 27 09:47:38 UTC 2016



> On Feb. 26, 2016, 5:13 p.m., Dāvis Mosāns wrote:
> > src/lib/activitiesmodel.h, line 96
> > <https://git.reviewboard.kde.org/r/127187/diff/1/?file=445509#file445509line96>
> >
> >     Wouldn't it be better to use _MSC_VER <= 1800 /* MSVC2013 */
> >     
> >     as maybe it works for MSVC2015?

Hm, thinking about this, what makes this tricky is that actually we are modifying the API/ABI of the class. I'm not too sure on the issues involved with mixing different versions of MSVC in the first place, but I think our safe options are either:
 a) Find a solution that works without adding any new members
 b) Keep the solution for _all_ versions of MSVC.
And in fact, spelling it out like this, option a) is clearly better. So back to the drawing board, consider this review request as on hold... Further ideas still welcome!


- Thomas


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


On Feb. 26, 2016, 9:32 a.m., Thomas Friedrichsmeier wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/127187/
> -----------------------------------------------------------
> 
> (Updated Feb. 26, 2016, 9:32 a.m.)
> 
> 
> Review request for KDE Frameworks, kdewin and Ivan Čukić.
> 
> 
> Repository: kactivities
> 
> 
> Description
> -------
> 
> MSVC (2013) complains that beginResetModel(), etc. are protected. This patch allows the code to compile. It is clearly not very elegant, but both my templating- and MSVC-foo is extremely limited. Not sure, if there is a better fix.
> 
> 
> Diffs
> -----
> 
>   src/lib/activitiesmodel.h 7258b73 
> 
> Diff: https://git.reviewboard.kde.org/r/127187/diff/
> 
> 
> Testing
> -------
> 
> Now compiles with MSVC 2013 on Windows.
> 
> 
> Thanks,
> 
> Thomas Friedrichsmeier
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160227/bc9911c9/attachment.html>


More information about the Kde-frameworks-devel mailing list