Code review comment for lp://staging/~michihenning/unity-scopes-api/qt-coverage

Revision history for this message
Xavi Garcia (xavi-garcia-mena) wrote :

Regarding the _ENABLE_QT_EXPERIMENTAL_ I added it explicitly to all the headers to make any potencial user completely aware that he/she must use the flag. I thought that "hiding" the flag into another header would possibly be less clear to the final user, but it's fine with me to use that approach instead.

« Back to merge proposal