Usage of new API, cleanups (reduced)

src/org/adblockplus/android/AboutDialog.java

Index: src/org/adblockplus/android/AboutDialog.java
diff --git a/src/org/adblockplus/android/AboutDialog.java b/src/org/adblockplus/android/AboutDialog.java
index 2a0219b274cfcd0eb74beb84387c15606d1ae405..fef99bcea1d79d0469c0e53179227e2fefe06fce 100644
--- a/src/org/adblockplus/android/AboutDialog.java
+++ b/src/org/adblockplus/android/AboutDialog.java
@@ -31,34 +31,34 @@ public class AboutDialog extends Dialog
 {
   private static Context context = null;
 
-  public AboutDialog(Context context)
+  public AboutDialog(final Context context)
   {
     super(context);
     AboutDialog.context = context;
   }
 
   @Override
-  public void onCreate(Bundle savedInstanceState)
+  public void onCreate(final Bundle savedInstanceState)
   {
-    requestWindowFeature(Window.FEATURE_NO_TITLE);
-    setContentView(R.layout.about);
+    this.requestWindowFeature(Window.FEATURE_NO_TITLE);

[Felix Dahlke, 2014/04/16 15:24:25]

As before, I'm fine with the "final" change, but not with the "this." one.

[René Jeschke, 2014/04/16 17:51:47]

Done.

+    this.setContentView(R.layout.about);
 
     // Get package version code and name
     String versionName = "--";
     int versionCode = -1;
     try
     {
-      PackageInfo pi = context.getPackageManager().getPackageInfo(context.getPackageName(), 0);
+      final PackageInfo pi = context.getPackageManager().getPackageInfo(context.getPackageName(), 0);
       versionName = pi.versionName;
       versionCode = pi.versionCode;
     }
-    catch (NameNotFoundException ex)
+    catch (final NameNotFoundException ex)
     {
       // ignore - it can not happen because we query information about ourselves
     }
 
     // Construct html
-    StringBuilder info = new StringBuilder();
+    final StringBuilder info = new StringBuilder();
     info.append("<h3>");
     info.append(context.getString(R.string.app_name));
     info.append("</h3>");
@@ -75,7 +75,7 @@ public class AboutDialog extends Dialog
     AdblockPlus.appendRawTextFile(context, info, R.raw.legal);
 
     // Show text
-    TextView tv = (TextView) findViewById(R.id.about_text);
+    final TextView tv = (TextView)this.findViewById(R.id.about_text);
     tv.setText(Html.fromHtml(info.toString()));
     tv.setMovementMethod(LinkMovementMethod.getInstance());
   }