Fixed a crash in a constructor of FullPrintConfig due to an incorrect use of virtual inheritance. Note that an invocation of ConfigBase::optptr() is routed to FullPrintConfig::optptr() for all classes of the FullPrintConfig hierarchy. FullPrintConfig::optptr() in turn invokes optptr() of PrintObjectConfig, PrintRegionConfig, PrintConfig and HostConfig. Due to the use of virtual inheritance, this all happens, when PrintObjectConfig gets constructed as part of FullPrintConfig, but at that time PrintRegionConfig, PrintConfig and HostConfig are not constructed yet. Accessing them at that time leads to crashes, when compiled with Visual Studio 2013 compiler. For some reason the code generated by gcc does not crash, but I believe the behavior is undefined and it is better to be fixed anyway.
Fixed a crash in a constructor of FullPrintConfig due to an incorrect use
of virtual inheritance. Note that an invocation of ConfigBase::optptr()
is routed to FullPrintConfig::optptr() for all classes of the FullPrintConfig
hierarchy. FullPrintConfig::optptr() in turn invokes optptr()
of PrintObjectConfig, PrintRegionConfig, PrintConfig and HostConfig.
Due to the use of virtual inheritance, this all happens, when
PrintObjectConfig gets constructed as part of FullPrintConfig, but
at that time PrintRegionConfig, PrintConfig and HostConfig are not
constructed yet. Accessing them at that time leads to crashes,
when compiled with Visual Studio 2013 compiler. For some reason
the code generated by gcc does not crash, but I believe the behavior
is undefined and it is better to be fixed anyway.

The patch solves the problem by calling set_defaults() by the topmost
object, which not only fixes the crashes, but also avoids repeated
initialization.

prusa3d/Slic3r,prusa3d/Slic3r,prusa3d/Slic3r,prusa3d/Slic3r,prusa3d/Slic3r,prusa3d/Slic3r,prusa3d/Slic3r