[Kde-games-devel] Review Request 111049: KNetWalk QML port

[Ashwin Rajeev]

> On Aug. 19, 2013, 10:33 p.m., Albert Astals Cid wrote:
> > From my "i don't know much about KNetWalk" point of view you can commit it, seems it does the same in less code, so for me it's fine. Maybe you should wait a few days in case anyone else has an opinion, and then commit it to master + blog (is your blog in planet.kde.org?) announcing the world the change and that we need some testers to make sure nothing regressed.

i don't have a blog :)


- Ashwin


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111049/#review38179
-----------------------------------------------------------


On Aug. 19, 2013, 8:27 p.m., Ashwin Rajeev wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111049/
> -----------------------------------------------------------
> 
> (Updated Aug. 19, 2013, 8:27 p.m.)
> 
> 
> Review request for KDE Games, Albert Astals Cid and Viranch Mehta.
> 
> 
> Description
> -------
> 
> QML port
> 
> 
> Diffs
> -----
> 
>   src/CMakeLists.txt d017d1a 
>   src/abstractgrid.h ddd9911 
>   src/cell.h 9254935 
>   src/cell.cpp 7555e1d 
>   src/fielditem.h 91e1a33 
>   src/fielditem.cpp a04ad6e 
>   src/main.cpp ce3d7a6 
>   src/mainwindow.h 2c047b6 
>   src/mainwindow.cpp d7d6fbd 
>   src/qml/Cable.qml PRE-CREATION 
>   src/qml/CanvasItem.qml PRE-CREATION 
>   src/qml/Cell.qml PRE-CREATION 
>   src/qml/logic.js PRE-CREATION 
>   src/qml/main.qml PRE-CREATION 
>   src/scene.h 7753303 
>   src/scene.cpp 38c276f 
>   src/view.h 9de36b8 
>   src/view.cpp 95fe9b4 
> 
> Diff: http://git.reviewboard.kde.org/r/111049/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Ashwin Rajeev
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-games-devel/attachments/20130820/807f1f50/attachment-0001.html>