diego1996 / gamekit

Automatically exported from code.google.com/p/gamekit
0 stars 0 forks source link

gkWindowAndroid.cpp has a bug with touchReleased event #236

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Engine/Android/gkWindowAndroid.cpp has a bug with the touchRelease method at 
line 204

bool gkWindowAndroid::touchReleased(const OIS::MultiTouchEvent& arg)
{
        gkMouse& data = m_mouse;

        data.buttons[gkMouse::Left] = GK_Released;

        if (!m_listeners.empty())
        {
                gkWindowSystem::Listener* node = m_listeners.begin();
                while (node)
                {
------------ node->mousePressed(data);
+++++++++ node->mouseReleased(data);
                        node = node->getNext();
                }
        }

        return true;
}

Original issue reported on code.google.com by kizan...@gmail.com on 1 Jun 2012 at 6:42

GoogleCodeExporter commented 9 years ago
I think it should call node->mouseReleased(data) instead of  
node->mousePressed(data);

Original comment by kizan...@gmail.com on 1 Jun 2012 at 6:44

GoogleCodeExporter commented 9 years ago
Thx for reporting. I changed it and it is in trunk now! Sry, it took a while 
but I didn't work on android lately.

Here is the commit:
http://code.google.com/p/gamekit/source/detail?r=1103

Original comment by thomas.t...@googlemail.com on 4 Jul 2012 at 5:06