Horde3D

Next-Generation Graphics Engine
It is currently 28.03.2024, 09:46

All times are UTC + 1 hour




Post new topic Reply to topic  [ 2 posts ] 
Author Message
PostPosted: 16.09.2010, 00:56 
Offline

Joined: 11.09.2010, 20:21
Posts: 44
Location: Germany
I just skimmed over the newest revision 551 with the new renderer abstraction and I think I spotted a bug. in Renderer::drawMeshes you call the new method RenderDeviceInterface::drawIndexed. This expects a "numVerts" argument which is also treated so (and thus added to the base vertex in glDrawRangeElements), but you call this function with the mesh's "vertREnd" property (instead the right "vertREnd-vertRStart+1").

The samples work correctly because the end vertex in glDrawRangeElements is just a hint (as long as it's too large) and it maybe no performance cost, but it is just the wrong usage.


Top
 Profile  
Reply with quote  
PostPosted: 16.09.2010, 20:15 
Offline
Tool Developer

Joined: 13.11.2007, 11:07
Posts: 1150
Location: Germany
Good spot. Thanks for reporting. Fixed it.


Top
 Profile  
Reply with quote  
Display posts from previous:  Sort by  
Post new topic Reply to topic  [ 2 posts ] 

All times are UTC + 1 hour


Who is online

Users browsing this forum: No registered users and 31 guests


You cannot post new topics in this forum
You cannot reply to topics in this forum
You cannot edit your posts in this forum
You cannot delete your posts in this forum
You cannot post attachments in this forum

Search for:
Jump to:  
cron
Powered by phpBB® Forum Software © phpBB Group