Am 15.07.2021 um 13:29 schrieb William F Pokorny:

> Looks like a problem with spline{} block parsing when passing a spline 
> id. The current Parse_Spline() code expects the spline type token to 
> exist otherwise it assumes the intent was a linear spline.
> 
> A fix is to add some dynamic_cast 'type checking' to:
> 
> GenericSpline *Parser::Parse_Spline()

Fortunately, there's a more elegant solution: The `GenericSpline` class 
has a virtual member function, `Clone()`, that is implemented by each 
subclass to return a newly constructed copy of whatever the actual type 
of the object is.

So the code in question can remain as simple as:

     if (!New)
     {
         if (Old)
             New = Old->Clone();
         else
             New = new LinearSpline();
     }

Will you do the honors, or shall I?

Post a reply to this message