Hi Jan,

2012/9/6 Jan Ciger <jan.ci...@gmail.com>

> Hello Rafael,
>
> On Wed, Sep 5, 2012 at 11:29 PM, Rafa Gaitan <rafa.gai...@gmail.com>wrote:
>
>> Hi Jan,
>>
>>
>>
>> Thank you, If it gives you any problem then send me the modified example
>> and I will try to reproduce it.
>>
>
> I have updated my code and the bug is still there.
>
> You can get my modified example here:
> https://dl.dropbox.com/u/6822332/org.openscenegraph.osgcamera.zip
>
> The data files are here (just unpack the folder to /sdcard on your device,
> it looks for files /sdcard/osg/*.ive):
> https://dl.dropbox.com/u/6822332/osg.zip
>
>

Thank you, there is indeed a bug there :S. I was able to reproduce it.  I
have also replicated it in native and it worked as expected, so something
is happening in the jni conversion part. I'm debugging now and I will keep
you informed.


>
> I've also checked in changes regarding the camera resolution I hope these
>> changes will also fix the problems on your side.
>>
>
> Yes, that fixed the camera-related crash.
>

Perfect!


>
>
> BTW, there is a syntax error in
> org.openscenegraph.android/jni/JNIosgViewer.cpp:
>
> @@ -138,7 +138,7 @@ JNIEXPORT void JNICALL
> Java_org_openscenegraph_osg_viewer_Viewer_nativeSetUpView
>
>  JNIEXPORT void JNICALL
> Java_org_openscenegraph_osg_viewer_Viewer_nativeSetViewport(JNIEnv *,
> jclass, jlong cptr, jint x, jint y, jint width, jint height)
>  {
> -    osgViewer::Viewer *v = reinterpret_cast<osgViewer::Viewer::Viewer*>
> (cptr);
> +    osgViewer::Viewer *v = reinterpret_cast<osgViewer::Viewer*> (cptr);
>      if (v == NULL)
>          return;
>      v->getEventQueue()->windowResize(x, y, width, height);


> The above makes it actually compile with the latest NDK from Google. You
> may also want to raise the minimum SDK level to 8 or 9 in the manifests,
> because Lint is complaining about functions being used that are not
> available in the old SDK.


Fixed and checked in the repository, thank you!

Maybe we should establish a submission protocol. I think git and gitorious
has something to easily merge changes from different clones. Something
called merge request, but I'm pretty new in GIT and I don't have clear the
complete process.

Regards,
Rafa.




>
>
> Jan
>
> _______________________________________________
> osg-users mailing list
> osg-users@lists.openscenegraph.org
> http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org
>
>


-- 
Rafael Gaitán Linares
CTO at Mirage Technologies S.L - http://www.mirage-tech.com
gvSIG3D Developer - http://gvsig3d.blogspot.com
_______________________________________________
osg-users mailing list
osg-users@lists.openscenegraph.org
http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org

Reply via email to