(2019-06-20, 23:03)Michael Horvath Wrote: Up until now I have been wrapping the model files (generated using L3P up until this point) inside an outer file with scene settings, camera, etc. This is a lot easier for me since I don't have to edit each model file when a wrapper gets edited, renamed or merged together with other wrappers. I have dozens of models, and a dozen or so wrappers, so this is not trivial. And each time I make a change to a model I have to push each file to GitHub/SourceForge.
OK, I'll wrap all of the #declare statements in #ifndefs.
(2019-06-20, 23:03)Michael Horvath Wrote: I think it would be better to set a variable to equal true or false versus deleting, commenting out, or undeffing a line each time you want to skip or unskip a camera or light source.
Changing this
#declare LDXSkipCamera = true;
to this
#undef LDXSkipCamera;
each time is more error prone and more typing.
I'm not going to change this, because I don't really agree, and your suggested change will change the behavior for everyone else. If you want to define your own camera, #declare LDXSkipCamera to anything, and the generated POV will omit defining a camera. If you want to use the defined camera, don't define LDXSkipCamera (or #undef it instead of setting it to false if you define it in multiple places). Similarly, I will add #ifndef (LDXSkipBackground) around the background definition, so you can define that (to any value) to have LDView's POV omit its background declaration.