Thanks for your answer, Xavier. > $NON-NLS is something specific to Eclipse that we do not use in Android. > Our convention for the android source code do not enforce the use of > this qualifier. > > As for importing java.lang.String, those 2 files have been created > automatically by aidl (see the files in aidl/) and the tool doesn't > care about unneeded imports and automatically put import statements > for all the used classes.
Ok, fair comment about the java.lang.String import. That said, since you are recommending to use Eclipse as the development environment, I think it would be benefitial for everybody if a few minutes taken before release to make sure it compiles with any possible settings the user may have in their Eclipse project. If that means cleaning up the generated code, I believe that should be undertaken. Otherwise you will end up with aggrevated users. Do you honestly expect the user having to modify their project warning levels to be able to successfully integrate LVL? Perhaphs the answer is yes, but then it should be documented and spelt out how warning levels need to be set for the project. -- You received this message because you are subscribed to the Google Groups "Android Developers" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/android-developers?hl=en
