SkiaSampleRenderer - handle case where compiler won't cast jint to void*.

platform_tools/android/app/jni/com_skia_SkiaSampleRenderer.cpp

 
 /*
  * Copyright 2011 Skia
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 #include "com_skia_SkiaSampleRenderer.h"
 
 #include "GrContext.h"
@@ skipping 236 matching lines @@
     gWindow->update(NULL);
 }
 
 JNIEXPORT void JNICALL Java_com_skia_SkiaSampleRenderer_updateSize(JNIEnv* env,
         jobject thiz, jint w, jint h)
 {
     gWindow->resize(w, h);
 }
 
 JNIEXPORT void JNICALL Java_com_skia_SkiaSampleRenderer_handleClick(JNIEnv* env,
         jobject thiz, jint owner, jfloat x, jfloat y, jint jstate)

[djsollen, 2014/04/17 17:43:50]

I also think we should consider updating the owner variable to jlong to better
future proof this.

 {
     SkView::Click::State state;
     switch(jstate) {
         case 0:     // MotionEvent.ACTION_DOWN
             state = SkView::Click::kDown_State;
             break;
         case 1:     // MotionEvent.ACTION_UP
         case 3:     // MotionEvent.ACTION_CANCEL
             state = SkView::Click::kUp_State;
             break;
         case 2:     // MotionEvent.ACTION_MOVE
             state = SkView::Click::kMoved_State;
             break;
         default:
             SkDebugf("motion event ignored\n");
             return;
     }
-    gWindow->handleClick(x, y, state, (void*) owner);
+    gWindow->handleClick(x, y, state, reinterpret_cast<void*>(owner));
 }
 
 JNIEXPORT void JNICALL Java_com_skia_SkiaSampleRenderer_nextSample(
         JNIEnv* env, jobject thiz)
 {
     gWindow->nextSample();
 }
 
 JNIEXPORT void JNICALL Java_com_skia_SkiaSampleRenderer_previousSample(
         JNIEnv* env, jobject thiz)
@@ skipping 61 matching lines @@
         JNIEnv* env, jobject thiz)
 {
     gWindow->saveToPdf();
 }
 
 JNIEXPORT void JNICALL Java_com_skia_SkiaSampleRenderer_postInval(
         JNIEnv* env, jobject thiz)
 {
     gWindow->postInvalDelay();
 }