What about the change above in setClientModel() should we define as coding-style default that all setModel() functions should include the NULL pointer check?