search

atmtools / arts

The Atmospheric Radiative Transfer Simulator
https://www.radiativetransfer.org/
Other
66 stars 30 forks source link

Put the wrapper for the WSV in a separate header seen only by a few cpp-files #814

Closed riclarsson closed 1 month ago

riclarsson commented 2 months ago

The idea is that there should be no way now to set a WsvValue directly. We want this because we want the dtor to not be created. If it is created, every file that somehow need to access auto_wsg.h is getting a dtor, or 4-5 seconds longer compilation time. This includes wokrspace.h, which is used extensively.

riclarsson commented 2 months ago

@olemke , can you run your script and see if the long dtor for the Wsv Variant is gone from the artsworkspace target on this branch?