Android: AbstractWizardModel bug

[Meeh]

java.lang.RuntimeException?:

at android.app.ActivityThread?.performLaunchActivity (ActivityThread?.java:2900)
at android.app.ActivityThread?.handleLaunchActivity (ActivityThread?.java:2979)
at android.app.ActivityThread?.handleRelaunchActivity (ActivityThread?.java:4868)
at android.app.ActivityThread?.-wrap18 (Unknown Source)
at android.app.ActivityThread?$H.handleMessage (ActivityThread?.java:1640)
at android.os.Handler.dispatchMessage (Handler.java:105)
at android.os.Looper.loop (Looper.java:164)
at android.app.ActivityThread?.main (ActivityThread?.java:6673)
at java.lang.reflect.Method.invoke (Native Method)
at com.android.internal.os.Zygote$MethodAndArgsCaller?.run (Zygote.java:240)
at com.android.internal.os.ZygoteInit?.main (ZygoteInit?.java:769)

Caused by: java.lang.NullPointerException?:

at net.i2p.android.wizard.model.AbstractWizardModel?.load (AbstractWizardModel?.java:70)
at net.i2p.android.wizard.ui.AbstractWizardActivity?.onCreate (AbstractWizardActivity?.java:52)
at android.app.Activity.performCreate (Activity.java:6984)
at android.app.Instrumentation.callActivityOnCreate (Instrumentation.java:1235)
at android.app.ActivityThread?.performLaunchActivity (ActivityThread?.java:2853)

[zzz]

I think you could fix this by making mRootPageList final.
Looks like mListeners and mContext could be final also.

[Zlatin Balevsky]

I'm not familiar with the threading model on Android, but I suggest we make net.i2p.android.wizard.model.PageList extend CopyOnWriteArrayList

[Meeh]

Expanded the code and hope this happens again so we can figure out where the NPE came from.

[zzz]

Due to a misunderstanding? My recommendation in comment 1 was not implemented for 38. Please implement it for .39.

[zzz]

finals checked in for 0.9.40 as described in comment 1