Reduce the number of redundant shaders

[Schneegans]

There are several classes which are instantiated multiple times with each instance having an identical shader program. This should be optimized! Here are some classes which are affected:

• WorldSpaceGuiArea.hpp
• ScreenSpaceGuiArea.hpp
• Trajectory.hpp
• Mark.hpp
• Sunflare.hpp
• DeepSpaceDot.hpp
• SimpleBody.hpp
• Sharad.hpp
• Ring.hpp
• PolygonTool.hpp
• PathTool.hpp
• EllipseTool.hpp
• DipStrikeTool.hpp

[JonasGilg]

Someone else can take the ticket for now. In csp-simple-bodies is already a branch, where I started work.

[JonasGilg]

@Schneegans Why is the Trajectory render logic inside cs-scene, instead of csp-trajectories?
Should I move it over?

[Schneegans]

It's kept in cs-scene since it may be of use for other plugins. I guess we should leave it the way it is for now.

[Schneegans]

I would suggest to move this issue to v1.3.0 together with #62 since it involves some intense refactoring and testing. What do you think?

[JonasGilg]

Yes, this sounds good.

[Schneegans]

What's the progress on this one? Do you have time to work on this in the near future? Else I would suggest to move this to the next milestone.

[JonasGilg]

What's the progress on this one?

Close to none. I think this should be done together with a rework of our render pipeline where we have more control, since we don't make much use of the scene graph.

Do you have time to work on this in the near future?

No, sadly not.

Else I would suggest to move this to the next milestone.

Yes we can do that.