Code review comment for lp:~rpadovani/ubuntu-weather-app/1200600

Revision history for this message
Martin Borho (martin-borho) wrote :

Thanks for the fix, that was fast!

But one request:

By using only "if (citiesModel.count < 1) {" in the onMessage handler instead of "if (!number) {", you could remove the "number" var. Leads to less code.

Cheers
Martin

review: Needs Fixing

« Back to merge proposal