dektar / ColorNamer

The code behind the ColorNamer app, https://play.google.com/store/apps/details?id=com.color.colornamer&hl=en
GNU General Public License v3.0
25 stars 41 forks source link

Crash when choosing a colour on Android 4.4 #3

Open dalb8 opened 10 years ago

dalb8 commented 10 years ago

I installed v2.3 on a Nexus 7 (2012) running Android 4.4 and when I go to the Chooser (Main Activity) and tap a colour, the app crashes. There is no such crash on another device running Android 4.2. BTW, the Help section still says v2.2.

E/MessageQueue-JNI(21065): at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:593) E/MessageQueue-JNI(21065): at dalvik.system.NativeStart.main(Native Method) D/AndroidRuntime(21065): Shutting down VM W/dalvikvm(21065): threadid=1: thread exiting with uncaught exception (group=0x419f5b90) E/AndroidRuntime(21065): FATAL EXCEPTION: main E/AndroidRuntime(21065): Process: com.color.colornamer, PID: 21065 E/AndroidRuntime(21065): java.lang.IllegalArgumentException: x must be < bitmap.width() E/AndroidRuntime(21065): at android.graphics.Bitmap.checkPixelAccess(Bitmap.java:1325) E/AndroidRuntime(21065): at android.graphics.Bitmap.getPixel(Bitmap.java:1277) E/AndroidRuntime(21065): at com.color.colornamer.ColorPickerView.getColor(ColorPickerView.java:105) E/AndroidRuntime(21065): at com.color.colornamer.MainActivity.onTouch(MainActivity.java:143) E/AndroidRuntime(21065): at android.view.View.dispatchTouchEvent(View.java:7685) E/AndroidRuntime(21065): at android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:2210) E/AndroidRuntime(21065): at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:1945) E/AndroidRuntime(21065): at android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:2216) E/AndroidRuntime(21065): at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:1917) E/AndroidRuntime(21065): at android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:2216) E/AndroidRuntime(21065): at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:1917) E/AndroidRuntime(21065): at android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:2216) E/AndroidRuntime(21065): at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:1917) E/AndroidRuntime(21065): at android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:2216) E/AndroidRuntime(21065): at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:1917) E/AndroidRuntime(21065): at android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:2216) E/AndroidRuntime(21065): at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:1917) E/AndroidRuntime(21065): at android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:2216) E/AndroidRuntime(21065): at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:1917) E/AndroidRuntime(21065): at com.android.internal.policy.impl.PhoneWindow$DecorView.superDispatchTouchEvent(PhoneWindow.java:2066) E/AndroidRuntime(21065): at com.android.internal.policy.impl.PhoneWindow.superDispatchTouchEvent(PhoneWindow.java:1515) E/AndroidRuntime(21065): at android.app.Activity.dispatchTouchEvent(Activity.java:2457) E/AndroidRuntime(21065): at com.android.internal.policy.impl.PhoneWindow$DecorView.dispatchTouchEvent(PhoneWindow.java:2014) E/AndroidRuntime(21065): at android.view.View.dispatchPointerEvent(View.java:7870) E/AndroidRuntime(21065): at android.view.ViewRootImpl$ViewPostImeInputStage.processPointerEvent(ViewRootImpl.java:3919) E/AndroidRuntime(21065): at android.view.ViewRootImpl$ViewPostImeInputStage.onProcess(ViewRootImpl.java:3808) E/AndroidRuntime(21065): at android.view.ViewRootImpl$InputStage.deliver(ViewRootImpl.java:3406) E/AndroidRuntime(21065): at android.view.ViewRootImpl$InputStage.onDeliverToNext(ViewRootImpl.java:3456) E/AndroidRuntime(21065): at android.view.ViewRootImpl$InputStage.forward(ViewRootImpl.java:3425) E/AndroidRuntime(21065): at android.view.ViewRootImpl$AsyncInputStage.forward(ViewRootImpl.java:3510) E/AndroidRuntime(21065): at android.view.ViewRootImpl$InputStage.apply(ViewRootImpl.java:3433) E/AndroidRuntime(21065): at android.view.ViewRootImpl$AsyncInputStage.apply(ViewRootImpl.java:3567) E/AndroidRuntime(21065): at android.view.ViewRootImpl$InputStage.deliver(ViewRootImpl.java:3406) E/AndroidRuntime(21065): at android.view.ViewRootImpl$InputStage.onDeliverToNext(ViewRootImpl.java:3456) E/AndroidRuntime(21065): at android.view.ViewRootImpl$InputStage.forward(ViewRootImpl.java:3425) E/AndroidRuntime(21065): at android.view.ViewRootImpl$InputStage.apply(ViewRootImpl.java:3433) E/AndroidRuntime(21065): at android.view.ViewRootImpl$InputStage.deliver(ViewRootImpl.java:3406) E/AndroidRuntime(21065): at android.view.ViewRootImpl.deliverInputEvent(ViewRootImpl.java:5520) E/AndroidRuntime(21065): at android.view.ViewRootImpl.doProcessInputEvents(ViewRootImpl.java:5500) E/AndroidRuntime(21065): at android.view.ViewRootImpl.enqueueInputEvent(ViewRootImpl.java:5471) E/AndroidRuntime(21065): at android.view.ViewRootImpl$WindowInputEventReceiver.onInputEvent(ViewRootImpl.java:5594) E/AndroidRuntime(21065): at android.view.InputEventReceiver.dispatchInputEvent(InputEventReceiver.java:182) E/AndroidRuntime(21065): at android.os.MessageQueue.nativePollOnce(Native Method) E/AndroidRuntime(21065): at android.os.MessageQueue.next(MessageQueue.java:137) E/AndroidRuntime(21065): at android.os.Looper.loop(Looper.java:124) E/AndroidRuntime(21065): at android.app.ActivityThread.main(ActivityThread.java:4998) E/AndroidRuntime(21065): at java.lang.reflect.Method.invokeNative(Native Method) E/AndroidRuntime(21065): at java.lang.reflect.Method.invoke(Method.java:515) E/AndroidRuntime(21065): at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:777) E/AndroidRuntime(21065): at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:593) E/AndroidRuntime(21065): at dalvik.system.NativeStart.main(Native Method) W/ActivityManager( 500): Force finishing activity com.color.colornamer/.MainActivity D/dalvikvm( 500): GC_FOR_ALLOC freed 1460K, 20% free 21341K/26424K, paused 74ms, total 74ms W/ActivityManager( 500): Activity pause timeout for ActivityRecord{426c32e0 u0 com.color.colornamer/.MainActivity t425 f}

dektar commented 10 years ago

I am unable to reproduce this on my v2.3 on a Nexus 7 (2012) running Android 4.4 (luckily I have one too!). I just re-uploaded directly, rather than downloading from the play store, and have no issues.