spacing seems off here, also you should move this up and have it be action(bla)->setEnabled(game.isValid())

spacing is also off.

You need to increase the version if you add stuff

also you should move this up and have it be action(bla)->setEnabled(game.isValid())

oh, yes. I'll do that.

spacing seems off here

This is because this file uses tabs instead of spaces. I would convert the file in a separate commit after the patch is accepted or should I do it at once?

Because of tabs, see above.

About spacing: my fault. Somehow the tab width in Kate was set to 8 characters instead of 4.

Don't change the spacing policy of a file, you use whatever spacing it has, changing it is usually not a good idea.

Other files of the project have the spaces instead of tabs policy. Shouldn't it be harmonized somehow?

Use now tabs for this.

Done. (This file contains a mixed policy of spaces and tabs: first element spaces, rest tabs)

The problem with doing that is that it makes git blame a bit harder to use :/

Yes, I understand. What about changing those lines to spaces which are touched by a patch anyway. Can I configure Kate or KDevelop to have a spacing policy per file?

Then the file looks like a frankenfile. I've no idea if you can configure different spacing per file other than changing it using the statusbar widgets on each file

ksudoku.cpp already looks like this: several code blocks have spaces whereas the most have tabs. Anyway, I'll change my spaces to tabs.

And thanks for the quick review :-)