On segunda-feira, 7 de outubro de 2013 22:11:28, Alejandro Exojo wrote:
> Hi.
> 
> I've stumbled upon the names of the classes in Qt Android Extras:
> - QJNIEnvironment
> - QJNIObject
> 
> The all capital part due to the acronym is inconsistent with other Qt
> classes, where the acronym is written as if it were a word: e.g. QHttp*
> instead of QHTTP*, QUrl instead of QURL, etc.
> 
> I've mentioned on IRC, and Rich Moore told me it wasn't too late to mention
> the issue. He said it might be possible that the naming was chosen to be
> consistent with JNIEnv.
> 
> What do you think?

Unless there's a compelling reason otherwise, I'd suggest renaming to QJniXxx.

Also, shouldn't there be an "Android" somewhere there?
-- 
Thiago Macieira - thiago.macieira (AT) intel.com
  Software Architect - Intel Open Source Technology Center

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
Development mailing list
[email protected]
http://lists.qt-project.org/mailman/listinfo/development

Reply via email to