address-book-app

Code review comment for lp:~renatofilho/address-book-app/dynamic-backend

  1. dynamic-backend
  2. Merge into trunk
Revision history for this message
Renato Araujo Oliveira Filho (renatofilho) wrote :

> 57 + // use galare as default QtContacts Manager
> 58 + if (qEnvironmentVariableIsSet("QTCONTACTS_MANAGER_OVERRIDE")) {
> 59 + defaultManager = qgetenv("QTCONTACTS_MANAGER_OVERRIDE");
> 60 + }
>
> Wouldn't it be a good idea to check if the value set from command line
> actually exists in the list of available managers?
If the manager does not exists it will use the default one.

« Back to merge proposal