633 2012-10-28 16:02:37 +0100 PMVMatrix/ProjectFloat broken 2012-10-31 23:11:35 +0100 1 1 3 JogAmp Jogl util 2 All all VERIFIED FIXED --- normal --- 0 thomas_de_bodt sgothel --- jogl f56e36bd4a497643756ab737422eb356b963b27d --- oldest_to_newest 1928 0 thomas_de_bodt 2012-10-28 16:02:37 +0100 It seems some of the function in PMVMatrix/ProjectFloat do not work correctly. Looking at the code the problem seems to be that the the ProjectFloat class assumes that the Buffers.slice2Float returns a buffer with position = 0. As a result some methods (ex. gluLookAt) do not behave correctly. I attached a unit test. Both tests should produce a different result but in fact they currently produce the same result (e.g. and identity matrix). 1929 1 380 thomas_de_bodt 2012-10-28 16:03:33 +0100 Created attachment 380 Unit test 1932 2 sgothel 2012-10-31 16:54:20 +0100 Thank you, fixed (added your unit test). Pls verify if possible. 1942 3 thomas_de_bodt 2012-10-31 23:11:35 +0100 Looks good: the lookAt now behaves as expected. Thanks for the quick fix! 380 2012-10-28 16:03:33 +0100 2012-10-28 16:03:33 +0100 Unit test PMVMatrixTest.java application/octet-stream 1823 thomas_de_bodt cGFja2FnZSBjb20uam9nYW1wLm9wZW5nbC51dGlsOw0KDQppbXBvcnQgb3JnLmp1bml0LkJlZm9y ZTsNCmltcG9ydCBvcmcuanVuaXQuVGVzdDsNCg0KaW1wb3J0IGphdmF4Lm1lZGlhLm9wZW5nbC5m aXhlZGZ1bmMuR0xNYXRyaXhGdW5jOw0KaW1wb3J0IGphdmEubmlvLkZsb2F0QnVmZmVyOw0KDQpp bXBvcnQgc3RhdGljIG9yZy5qdW5pdC5Bc3NlcnQuYXNzZXJ0QXJyYXlFcXVhbHM7DQoNCi8qKg0K ICogQGF1dGhvciBUaG9tYXMgRGUgQm9kdA0KICovDQpwdWJsaWMgY2xhc3MgUE1WTWF0cml4VGVz dCB7DQoNCiAgcHJpdmF0ZSBQTVZNYXRyaXggZk1hdDsNCg0KICBAQmVmb3JlDQogIHB1YmxpYyB2 b2lkIHNldFVwKCkgdGhyb3dzIEV4Y2VwdGlvbiB7DQogICAgZk1hdCA9IG5ldyBQTVZNYXRyaXgo KTsNCiAgfQ0KDQogIEBUZXN0DQogIHB1YmxpYyB2b2lkIHRlc3RMb29rQXROZWdaSXNOb09wKCkg dGhyb3dzIEV4Y2VwdGlvbiB7DQogICAgZk1hdC5nbE1hdHJpeE1vZGUoR0xNYXRyaXhGdW5jLkdM X01PREVMVklFVyk7DQogICAgLy8gTG9vayB0b3dhcmRzIC16DQogICAgZk1hdC5nbHVMb29rQXQo DQogICAgICAgIDAsIDAsIDAsDQogICAgICAgIDAsIDAsIC0xLA0KICAgICAgICAwLCAxLCAwDQog ICAgKTsNCiAgICBGbG9hdEJ1ZmZlciBtdk1hdHJpeCA9IGZNYXQuZ2xHZXRNdk1hdHJpeGYoKTsN CiAgICBmbG9hdFtdIG12TWF0cml4QXJyID0gbmV3IGZsb2F0WzE2XTsNCiAgICBtdk1hdHJpeC5h c1JlYWRPbmx5QnVmZmVyKCkuZ2V0KG12TWF0cml4QXJyKTsNCiAgICBhc3NlcnRBcnJheUVxdWFs cygNCiAgICAgICAgLyoqDQogICAgICAgICAqIFRoZSAzIHJvd3Mgb2YgdGhlIG1hdHJpeCAoPSB0 aGUgMyBjb2x1bW5zIG9mIHRoZSBhcnJheS9idWZmZXIpIHNob3VsZCBiZTogc2lkZSwgdXAsIC1m b3J3YXJkLg0KICAgICAgICAgKi8NCiAgICAgICAgbmV3IGZsb2F0W10gew0KICAgICAgICAgICAg MSwgMCwgMCwgMCwNCiAgICAgICAgICAgIDAsIDEsIDAsIDAsDQogICAgICAgICAgICAwLCAwLCAx LCAwLA0KICAgICAgICAgICAgMCwgMCwgMCwgMQ0KICAgICAgICB9LA0KICAgICAgICBtdk1hdHJp eEFyciwNCiAgICAgICAgMWUtNmYNCiAgICApOw0KICB9DQogIEBUZXN0DQogIHB1YmxpYyB2b2lk IHRlc3RMb29rQXRQb3NZKCkgdGhyb3dzIEV4Y2VwdGlvbiB7DQogICAgZk1hdC5nbE1hdHJpeE1v ZGUoR0xNYXRyaXhGdW5jLkdMX01PREVMVklFVyk7DQogICAgLy8gTG9vayB0b3dhcmRzICt5DQog ICAgZk1hdC5nbHVMb29rQXQoDQogICAgICAgIDAsIDAsIDAsDQogICAgICAgIDAsIDEsIDAsDQog ICAgICAgIDAsIDAsIDENCiAgICApOw0KICAgIEZsb2F0QnVmZmVyIG12TWF0cml4ID0gZk1hdC5n bEdldE12TWF0cml4ZigpOw0KICAgIGZsb2F0W10gbXZNYXRyaXhBcnIgPSBuZXcgZmxvYXRbMTZd Ow0KICAgIG12TWF0cml4LmFzUmVhZE9ubHlCdWZmZXIoKS5nZXQobXZNYXRyaXhBcnIpOw0KICAg IGFzc2VydEFycmF5RXF1YWxzKA0KICAgICAgICAvKioNCiAgICAgICAgICogVGhlIDMgcm93cyBv ZiB0aGUgbWF0cml4ICg9IHRoZSAzIGNvbHVtbnMgb2YgdGhlIGFycmF5L2J1ZmZlcikgc2hvdWxk IGJlOiBzaWRlLCB1cCwgLWZvcndhcmQuDQogICAgICAgICAqLw0KICAgICAgICBuZXcgZmxvYXRb XSB7DQogICAgICAgICAgICAxLCAwLCAwLCAwLA0KICAgICAgICAgICAgMCwgMCwgLTEsIDAsDQog ICAgICAgICAgICAwLCAxLCAwLCAwLA0KICAgICAgICAgICAgMCwgMCwgMCwgMQ0KICAgICAgICB9 LA0KICAgICAgICBtdk1hdHJpeEFyciwNCiAgICAgICAgMWUtNmYNCiAgICApOw0KICB9DQp9DQo=