Review Request: Achievements 8: Make use of the AchievementsManager and the ProjectMetaData classes.
Laszlo Papp
lpapp at kde.org
Sat Aug 27 21:45:08 UTC 2011
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/102245/#review6079
-----------------------------------------------------------
player/lib/models/gameviewitem.h
<http://git.reviewboard.kde.org/r/102245/#comment5366>
Why are these methods explicit ?
projectFilePath, if you are making this kind of changes.
player/lib/models/gameviewitem.cpp
<http://git.reviewboard.kde.org/r/102245/#comment5369>
projectFilePath, if you are making this kind of changes.
player/lib/models/gameviewitem.cpp
<http://git.reviewboard.kde.org/r/102245/#comment5370>
projectFilePath, if you are making this kind of changes.
player/lib/models/gameviewitem.cpp
<http://git.reviewboard.kde.org/r/102245/#comment5367>
If you are already changing, you can call it directly projectFilePath
player/lib/models/gameviewitem.cpp
<http://git.reviewboard.kde.org/r/102245/#comment5368>
projectFilePath accordingly
- Laszlo
On Aug. 27, 2011, 7:53 p.m., Felix Rohrbach wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/102245/
> -----------------------------------------------------------
>
> (Updated Aug. 27, 2011, 7:53 p.m.)
>
>
> Review request for Gluon.
>
>
> Summary
> -------
>
> Now, the user should be able to create games with simple statistics/achievements.
>
>
> Diffs
> -----
>
> engine/game.cpp 9495909
> engine/gameproject.cpp 1fc2eb4
> engine/projectmetadata.h a8b3758
> engine/projectmetadata.cpp 3b36771
> player/kdeext/gamedetailsoverlay.h 99923a2
> player/kdeext/gamedetailsoverlay.cpp b6f525c
> player/kdeext/gamesoverlay.cpp d468186
> player/lib/models/achievementsmodel.h 3df946e
> player/lib/models/achievementsmodel.cpp e1ebad0
> player/lib/models/commentsmodel.cpp 4572f23
> player/lib/models/gameitemsmodel.h d2e3604
> player/lib/models/gameitemsmodel.cpp c4c977c
> player/lib/models/gameviewitem.h a017862
> player/lib/models/gameviewitem.cpp 64dd46a
>
> Diff: http://git.reviewboard.kde.org/r/102245/diff
>
>
> Testing
> -------
>
>
> Thanks,
>
> Felix
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/gluon/attachments/20110827/5537a115/attachment.html>
More information about the Gluon
mailing list