commit a2ada0cb60d32819541920778024014274560c6f
author Jeongik Cha <jeongik@google.com> Sat Nov 17 15:11:45 2018 +0900
committer Jeongik Cha <jeongik@google.com> Sat Nov 17 06:51:47 2018 +0000
tree 64a96129ef56ab89d4fef0aef2306be3afba845d
parent 47fb0d6b08b44f9bd7f957cec001c81e110e5d48

prevent inheritance of writeToParcel, readFromParcel

To prevent overriding writeToParcel and readFromParcel, make them final in cpp and java
because ndk parcelable is not virtual, it doesn't need to be final

Bug: 118246983
Test: ./runtests.sh
Test: m -j
Change-Id: I53fb443fdf3fe3c404d844f8fe6992c0c737c9f1

generate_java.cpp

diff --git a/generate_java.cpp b/generate_java.cpp
index 1bf26c3..f09ac06 100644
--- a/generate_java.cpp
+++ b/generate_java.cpp
@@ -134,7 +134,7 @@
       new Variable(new Type(types, "android.os.Parcel", 0, false), "_aidl_parcel");
 
   Method* write_method = new Method;
-  write_method->modifiers = PUBLIC | OVERRIDE;
+  write_method->modifiers = PUBLIC | OVERRIDE | FINAL;
   write_method->returnType = new Type(types, "void", 0, false);
   write_method->name = "writeToParcel";
   write_method->parameters.push_back(parcel_variable);
@@ -158,7 +158,7 @@
   parcel_class->elements.push_back(write_method);
 
   Method* read_method = new Method;
-  read_method->modifiers = PUBLIC;
+  read_method->modifiers = PUBLIC | FINAL;
   read_method->returnType = new Type(types, "void", 0, false);
   read_method->name = "readFromParcel";
   read_method->parameters.push_back(parcel_variable);