Review Request 120104: remove 'override' keyword to make kdev-python (frameworks branch) build properly

Miquel Sabaté Solà mikisabate at gmail.com
Mon Sep 8 20:54:29 UTC 2014



> On Sept. 8, 2014, 8:04 p.m., Miquel Sabaté Solà wrote:
> > IIRC, we're targeting GCC 4.7 and Clang 3.1 (since the release of KDevelop 4.7). In these two cases this should compile just fine, so I don't know why you would want to remove that override :/
> 
> Laurent Navet wrote:
>     I have gcc 4.9.1, and it don't build with :)

gcc 4.9.1 here and it builds just fine :P Are you on the last commit of KDevplatform master. Keep in mind that a recent commit [1] on KDevplatform changed the type of the first argument.

[1] http://quickgit.kde.org/?p=kdevplatform.git&a=commit&h=cd15efed451a41ac2443812698381a8c4b88bcbe


- Miquel


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


On Sept. 8, 2014, 7:47 p.m., Laurent Navet wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/120104/
> -----------------------------------------------------------
> 
> (Updated Sept. 8, 2014, 7:47 p.m.)
> 
> 
> Review request for KDevelop and Sven Brauch.
> 
> 
> Repository: kdev-python
> 
> 
> Description
> -------
> 
> remove 'override' keyword to make kdev-python (frameworks branch) build properly
> 
> 
> Diffs
> -----
> 
>   codecompletion/items/functiondeclaration.h 2e301a5 
> 
> Diff: https://git.reviewboard.kde.org/r/120104/diff/
> 
> 
> Testing
> -------
> 
> just build.
> 
> 
> Thanks,
> 
> Laurent Navet
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20140908/20a70a8b/attachment.html>


More information about the KDevelop-devel mailing list