<table><tr><td style="">ltoscano requested changes to this revision.<br />ltoscano added a comment.<br />This revision now requires changes to proceed.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D5544" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>There are few errors; following the chat on IRC, I think you need a development environment:</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">/mnt/kde/git/kde/kdegames/ksudoku/src/gui/ksudoku.cpp: In member function ‘void KSudoku::updateShapesList()’:
/mnt/kde/git/kde/kdegames/ksudoku/src/gui/ksudoku.cpp:220:72: error: no matching function for call to ‘ksudoku::CustomGame::CustomGame(QString&, QString&, ksudoku::GameVariantCollection*&)’
   variant = new CustomGame(variantName, variantDataPath, m_gameVariants);
                                                                        ^
In file included from /mnt/kde/git/kde/kdegames/ksudoku/src/gui/ksudoku.cpp:73:0:
/mnt/kde/git/kde/kdegames/ksudoku/src/gui/gamevariants.h:170:2: note: candidate: ksudoku::CustomGame::CustomGame(const QString&, const QUrl&, ksudoku::GameVariantCollection*)
  CustomGame(const QString& name, const QUrl& url, GameVariantCollection* collection=0);
  ^~~~~~~~~~
/mnt/kde/git/kde/kdegames/ksudoku/src/gui/gamevariants.h:170:2: note:   no known conversion for argument 2 from ‘QString’ to ‘const QUrl&’
/mnt/kde/git/kde/kdegames/ksudoku/src/gui/gamevariants.h:168:7: note: candidate: ksudoku::CustomGame::CustomGame(const ksudoku::CustomGame&)
 class CustomGame : public GameVariant {
       ^~~~~~~~~~
/mnt/kde/git/kde/kdegames/ksudoku/src/gui/gamevariants.h:168:7: note:   candidate expects 1 argument, 3 provided
/mnt/kde/git/kde/kdegames/ksudoku/src/gui/gamevariants.h:168:7: note: candidate: ksudoku::CustomGame::CustomGame(ksudoku::CustomGame&&)
/mnt/kde/git/kde/kdegames/ksudoku/src/gui/gamevariants.h:168:7: note:   candidate expects 1 argument, 3 provided
/mnt/kde/git/kde/kdegames/ksudoku/src/gui/ksudoku.cpp: In member function ‘virtual void KSudoku::dragEnterEvent(QDragEnterEvent*)’:
/mnt/kde/git/kde/kdegames/ksudoku/src/gui/ksudoku.cpp:631:37: error: request for member ‘hasUrls’ in ‘event->QDragEnterEvent::<anonymous>.QDragMoveEvent::<anonymous>.QDropEvent::mimeData()’, which is of pointer type ‘const QMimeData*’ (maybe you meant to use ‘->’ ?)
     event->accept(event->mimeData().hasUrls());
                                     ^~~~~~~
/mnt/kde/git/kde/kdegames/ksudoku/src/gui/ksudoku.cpp: In member function ‘virtual void KSudoku::dropEvent(QDropEvent*)’:
/mnt/kde/git/kde/kdegames/ksudoku/src/gui/ksudoku.cpp:636:43: error: conversion from ‘const QMimeData*’ to non-scalar type ‘const QMimeData’ requested
     const QMimeData data = event->mimeData();
                            ~~~~~~~~~~~~~~~^~</pre></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R417 KSudoku</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D5544" rel="noreferrer">https://phabricator.kde.org/D5544</a></div></div><br /><div><strong>To: </strong>ouwerkerk, KDE Games, ltoscano<br /><strong>Cc: </strong>ltoscano, KDE Games<br /></div>