Is phabricator being silly or are your patches missing some stuff?

Like it says
...\nPlease install the
and ends there

In D9805#189130, @aacid wrote:

Is phabricator being silly or are your patches missing some stuff?

Like it says
...\nPlease install the
and ends there

Meh, my diff has those in the next line. The line ends " KMahjongg library.", Prefs::tileSet()));"

I just realzied this is not how you use QMEsasgebox critical.

Since it's static you just do QMessageBox::critical(...)

Second one should be Prefs::background and not Prefs::tileSet?

In D9805#189154, @aacid wrote:

I just realzied this is not how you use QMEsasgebox critical.

Since it's static you just do QMessageBox::critical(...)

Thanks for pointing that out.

Second one should be Prefs::background and not Prefs::tileSet?

Of course. Copy&Paste error. -_-

Thanks for the review. I will upload an updated patch tomorrow. :)