================ Comment at: test/libcxx/test/remote.py:223 @@ +222,3 @@ + # For temp cleanup + self.remote_temp_files = [] + self.remote_temp_dirs = [] ---------------- jroelofs wrote: > danalbert wrote: > > Your naming is inconsistent. `self.scpCommand` but > > `self.remote_temp_files`. I prefer PEP8 stlye (with underscores), but LIT > > already chose the anti-Python style, so let's try to stick with that. (I'm > > guilty of this crime too.) > as in camelCase for vars, and underscore_separated_functions() ? Ok. Actually, looking at lit's Test.py, it looks like the format is underscores for variables, camel-case for functions.
http://reviews.llvm.org/D7380 EMAIL PREFERENCES http://reviews.llvm.org/settings/panel/emailpreferences/ _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
