New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove DrawPoints & DrawLines methods #3303
Remove DrawPoints & DrawLines methods #3303
Conversation
…of DrawLines methods.
…s, Bug Fix Drawables::Points
… call the Draw method.
Should we organise a crowdfunding campaign to buy you lots and lots of |
I wouldn't object too strenuously but my girlfriend already complains about all my junk :D |
My girlfriend complains about my junk too. Please feel free to use that out of context |
Bit optimisation, bit refactoring, bit code cleanup. It avoids doing conversions and creating/destroying buffers so should run a bit faster. Also it removes the old interfaces in favour of |
Oh and it does kind of bug fix one thing, points were all being drawn at size |
Happy 2015. I see you've been busy doing good things. I'm testing this. I have some problems. Some exiting the program without messages, and others like this. *** glibc detected *** src/./pioneersp: double free or corruption (out): 0x0000000025251110 *** This can have relationship with this PR?0a87000 08:0c 3690452 /media/auxiliar/Building/APIONEER64/pioneersp/src/pioneersp |
Other: pioneersp: opengl/VertexBufferGL.cpp:61: Graphics::OGL::VertexBuffer::VertexBuffer(const Graphics::VertexBufferDesc&): Assertion `m_desc.numVertices > 0' failed. |
The view of the Scanner has been ok with this PR. Missing fix the size of the stars in background. Very small, since the implementation of GL3 |
…nsformed into position per draw call.
Fixed a bug with the |
Ah, good catch @walterar, thanks. |
I have more, but I tell you one by one. :) |
Keep 'em coming that way I can tackle them all at once :) they might have things in common! |
Yes I've found a few bugs. Really dumb code in some cases, all mine I might add! :D |
If you've managed to lay a big white elephant in a mini cooper, no impossible task for you. :)) |
…VertexBuffer under a number of conditions. Don't not backface cull points.
Right, that should fix the worst of the stupid mistakes. |
What's wrong with that? I think I'll have to go back a while to find what it used to look like but they weren't supposed to be rendering as tiny points this is actually closer to the sizes that were always being requested. I do agree however that it looks a bit crap ;) |
Reminds me minecraft ;) |
Scanner blobs are now square and are more easy to see, at least at my end. I don't know if that's a bug or feature. I wonder if #2834 is affected by this? |
Yes blobs have been missing for a while now, although I hadn't noticed #2834 myself! Any objections to me merging this soon (later today)? |
Fine by me |
Remove DrawPoints & DrawLines methods
@fluffyfreak opengl/VertexBufferGL.cpp:61: Graphics::OGL::VertexBuffer::VertexBuffer(const Graphics::VertexBufferDesc&): Assertion `m_desc.numVertices > 0' failed. Is relevant to this PR? |
Where / When do you get that? I've run it in debug and don't get it anywhere that I can find. |
This sometimes occurs when attempting to retrieve a recently saved game. It is likely that the problem is in the serializer. |
This removes the
DrawPoints
andDrawLines
methods from the renderer replacing all of the calls to them with a newDrawables::Points
class and the existingDrawables::Lines
class.It affects the Scanner widget, the SectorView, targetting crosshairs, SystemView selection boxes, the GalacticView "You Are here" green indicator, HudTrails, SpeedLines, TextEntry and probably a few other bits I've forgotten to list.
I.e: everything it changes should be quite visible, so if you would like to test then please take a look and compare how it used to look to how it currently does.
I've tried to do this as I've gone along but some of the changes affect a number of areas and I might have missed them.
On some platforms / combinations of hardware / drivers it might improve performance due to it reusing buffers instead of allocating and releasing them constantly but this is difficult to measure as I don't have a wide enough range of equipment to test on.
Andy