diff --git a/components/debug/debugdraw.cpp b/components/debug/debugdraw.cpp index 1180ceb513..6ee165c7ef 100644 --- a/components/debug/debugdraw.cpp +++ b/components/debug/debugdraw.cpp @@ -225,12 +225,34 @@ static int getIdexBufferWriteFromFrame(const long long int& nFrame) namespace Debug { + static void makeLineInstance(osg::Geometry& lines) + { + auto vertices = new osg::Vec3Array; + auto color = new osg::Vec3Array; + lines.setDataVariance(osg::Object::STATIC); + lines.setUseVertexArrayObject(true); + lines.setUseDisplayList(false); + lines.setCullingActive(false); + + lines.setVertexArray(vertices); + lines.setNormalArray(color, osg::Array::BIND_PER_VERTEX); + + lines.addPrimitiveSet(new osg::DrawArrays(osg::PrimitiveSet::LINES, 0, vertices->size())); + } + + + DebugCustomDraw::DebugCustomDraw() + { + mLinesToDraw = new osg::Geometry(); + makeLineInstance(*mLinesToDraw); + } + void DebugCustomDraw::drawImplementation(osg::RenderInfo& renderInfo) const { auto state = renderInfo.getState(); osg::GLExtensions* ext = osg::GLExtensions::Get(state->getContextID(), true); - const osg::StateSet* stateSet = this->getStateSet(); + const osg::StateSet* stateSet = getStateSet(); auto program = static_cast(stateSet->getAttribute(osg::StateAttribute::PROGRAM)); const osg::Program::PerContextProgram* pcp = program->getPCP(*state); @@ -271,13 +293,13 @@ namespace Debug switch (shapeToDraw.mDrawShape) { case DrawShape::Cube: - this->mCubeGeometry->drawImplementation(renderInfo); + mCubeGeometry->drawImplementation(renderInfo); break; case DrawShape::Cylinder: - this->mCylinderGeometry->drawImplementation(renderInfo); + mCylinderGeometry->drawImplementation(renderInfo); break; case DrawShape::WireCube: - this->mWireCubeGeometry->drawImplementation(renderInfo); + mWireCubeGeometry->drawImplementation(renderInfo); break; } } @@ -286,36 +308,6 @@ namespace Debug static_cast(mLinesToDraw->getNormalArray())->clear(); } - struct DebugLines - { - - static void makeLineInstance(osg::Geometry& lines) - { - auto vertices = new osg::Vec3Array; - auto color = new osg::Vec3Array; - - lines.setUseVertexArrayObject(true); - lines.setUseDisplayList(false); - lines.setCullingActive(false); - - lines.setVertexArray(vertices); - lines.setNormalArray(color, osg::Array::BIND_PER_VERTEX); - - lines.addPrimitiveSet(new osg::DrawArrays(osg::PrimitiveSet::LINES, 0, vertices->size())); - } - - DebugLines() - { - mLinesGeom[0] = new osg::Geometry(); - mLinesGeom[1] = new osg::Geometry(); - - makeLineInstance(*mLinesGeom[0]); - makeLineInstance(*mLinesGeom[1]); - } - - std::array, 2> mLinesGeom; - }; - class DebugDrawCallback : public SceneUtil::NodeCallback { public: @@ -325,9 +317,9 @@ namespace Debug { mDebugDrawer.mCurrentFrame = nv->getTraversalNumber(); int indexRead = getIdexBufferReadFromFrame(mDebugDrawer.mCurrentFrame); - auto& lines = mDebugDrawer.mDebugLines; - lines->mLinesGeom[indexRead]->removePrimitiveSet(0, 1); - lines->mLinesGeom[indexRead]->addPrimitiveSet(new osg::DrawArrays(osg::PrimitiveSet::LINES, 0, static_cast(lines->mLinesGeom[indexRead]->getVertexArray())->size())); + auto& lines = mDebugDrawer.mCustomDebugDrawer[indexRead]->mLinesToDraw; + lines->removePrimitiveSet(0, 1); + lines->addPrimitiveSet(new osg::DrawArrays(osg::PrimitiveSet::LINES, 0, static_cast(lines->getVertexArray())->size())); nv->pushOntoNodePath(mDebugDrawer.mCustomDebugDrawer[indexRead]); nv->apply(*mDebugDrawer.mCustomDebugDrawer[indexRead]); @@ -338,14 +330,13 @@ namespace Debug }; } -Debug::DebugDrawer::DebugDrawer(Shader::ShaderManager& shaderManager, osg::ref_ptr parentNode) +Debug::DebugDrawer::DebugDrawer(Shader::ShaderManager& shaderManager, osg::ref_ptr parentNode) : mParentNode(parentNode) { mCurrentFrame = 0; auto vertexShader = shaderManager.getShader("debug_vertex.glsl", Shader::ShaderManager::DefineMap(), osg::Shader::Type::VERTEX); auto fragmentShader = shaderManager.getShader("debug_fragment.glsl", Shader::ShaderManager::DefineMap(), osg::Shader::Type::FRAGMENT); auto program = shaderManager.getProgram(vertexShader, fragmentShader); - mDebugLines = std::make_unique(); mDebugDrawSceneObjects = new osg::Group; mDebugDrawSceneObjects->setCullingActive(false); @@ -375,9 +366,9 @@ Debug::DebugDrawer::DebugDrawer(Shader::ShaderManager& shaderManager, osg::ref_p wireCube->setUseVertexBufferObjects(true); generateWireCube(*wireCube, 1.); - for (std::size_t i = 0; i < mShapesToDraw.size(); i++) + for (std::size_t i = 0; i < mCustomDebugDrawer.size(); i++) { - mCustomDebugDrawer[i] = new DebugCustomDraw(mShapesToDraw[i], mDebugLines->mLinesGeom[i]); + mCustomDebugDrawer[i] = new DebugCustomDraw(); mCustomDebugDrawer[i]->setStateSet(stateset); mCustomDebugDrawer[i]->mWireCubeGeometry = wireCube; mCustomDebugDrawer[i]->mCubeGeometry = cubeGeometry; @@ -385,16 +376,17 @@ Debug::DebugDrawer::DebugDrawer(Shader::ShaderManager& shaderManager, osg::ref_p } mDebugDrawSceneObjects->addCullCallback(new DebugDrawCallback(*this)); - parentNode->addChild(mDebugDrawSceneObjects); + mParentNode->addChild(mDebugDrawSceneObjects); } Debug::DebugDrawer::~DebugDrawer() { -} + mParentNode->removeChild(mDebugDrawSceneObjects); +} void Debug::DebugDrawer::drawCube(osg::Vec3f mPosition, osg::Vec3f mDims, osg::Vec3f mColor) { - mShapesToDraw[getIdexBufferWriteFromFrame(this->mCurrentFrame)].push_back({ mPosition, mDims, mColor, DrawShape::Cube }); + mCustomDebugDrawer[getIdexBufferWriteFromFrame(mCurrentFrame)]->mShapesToDraw.push_back({ mPosition, mDims, mColor, DrawShape::Cube }); } void Debug::DebugDrawer::drawCubeMinMax(osg::Vec3f min, osg::Vec3f max, osg::Vec3f color) @@ -406,14 +398,15 @@ void Debug::DebugDrawer::drawCubeMinMax(osg::Vec3f min, osg::Vec3f max, osg::Vec void Debug::DebugDrawer::addDrawCall(const DrawCall& draw) { - mShapesToDraw[getIdexBufferWriteFromFrame(this->mCurrentFrame)].push_back(draw); + mCustomDebugDrawer[getIdexBufferWriteFromFrame(mCurrentFrame)]->mShapesToDraw.push_back(draw); } void Debug::DebugDrawer::addLine(const osg::Vec3& start, const osg::Vec3& end, const osg::Vec3 color) { - const int indexWrite = getIdexBufferWriteFromFrame(this->mCurrentFrame); - auto vertices = static_cast(mDebugLines->mLinesGeom[indexWrite]->getVertexArray()); - auto colors = static_cast(mDebugLines->mLinesGeom[indexWrite]->getNormalArray()); + const int indexWrite = getIdexBufferWriteFromFrame(mCurrentFrame); + const auto& lines = mCustomDebugDrawer[indexWrite]->mLinesToDraw; + auto vertices = static_cast(lines->getVertexArray()); + auto colors = static_cast(lines->getNormalArray()); vertices->push_back(start); vertices->push_back(end); diff --git a/components/debug/debugdraw.hpp b/components/debug/debugdraw.hpp index dcf9132712..842a1ab753 100644 --- a/components/debug/debugdraw.hpp +++ b/components/debug/debugdraw.hpp @@ -58,20 +58,18 @@ namespace Debug class DebugCustomDraw : public osg::Drawable { public: - DebugCustomDraw(std::vector& cubesToDraw, osg::ref_ptr& linesToDraw) : mShapesToDraw(cubesToDraw), mLinesToDraw(linesToDraw) {} + DebugCustomDraw(); - std::vector& mShapesToDraw; - osg::ref_ptr& mLinesToDraw; + mutable std::vector mShapesToDraw; + osg::ref_ptr mLinesToDraw; osg::ref_ptr mCubeGeometry; osg::ref_ptr mCylinderGeometry; osg::ref_ptr mWireCubeGeometry; - virtual void drawImplementation(osg::RenderInfo&) const; + virtual void drawImplementation(osg::RenderInfo&) const override; }; - struct DebugLines; - struct DebugDrawer { friend DebugDrawCallback; @@ -85,13 +83,11 @@ namespace Debug void addLine(const osg::Vec3& start, const osg::Vec3& end, const osg::Vec3 color = colorWhite); private: - std::unique_ptr mDebugLines; - - std::array, 2> mShapesToDraw; long long int mCurrentFrame; std::array, 2> mCustomDebugDrawer; osg::ref_ptr mDebugDrawSceneObjects; + osg::ref_ptr mParentNode; }; } #endif // !