diff --git a/apps/graph/cartesian_function.cpp b/apps/graph/cartesian_function.cpp index cccc14b30..9c6b307a8 100644 --- a/apps/graph/cartesian_function.cpp +++ b/apps/graph/cartesian_function.cpp @@ -23,40 +23,35 @@ void CartesianFunction::setDisplayDerivative(bool display) { } double CartesianFunction::approximateDerivative(double x, Poincare::Context * context) const { - Poincare::Expression * abscissa = new Poincare::Approximation(x); - Poincare::Expression * args[2] = {expression(context)->clone(), abscissa}; - Poincare::Derivative derivative(args, false); // derivative takes ownership of abscissa and the clone of expression - /* TODO: when we will simplify derivative, we might want to simplify the + Poincare::Derivative derivative(expression(context).clone(), Poincare::Float(x)); // derivative takes ownership of Poincare::Float(x) and the clone of expression + /* TODO: when we will approximate derivative, we might want to simplify the * derivative here. However, we might want to do it once for all x (to avoid * lagging in the derivative table. */ - return PoincareHelpers::ApproximateToScalar(&derivative, *context); + return PoincareHelpers::ApproximateToScalar(derivative, *context); } double CartesianFunction::sumBetweenBounds(double start, double end, Poincare::Context * context) const { - Poincare::Expression * x = new Poincare::Approximation(start); - Poincare::Expression * y = new Poincare::Approximation(end); - Poincare::Expression * args[3] = {expression(context)->clone(), x, y}; - Poincare::Integral integral(args, false); // Integral takes ownership of args - /* TODO: when we will simplify integral, we might want to simplify the + Poincare::Integral integral(expression(context).clone(), Poincare::Float(start), Poincare::Float(end)); // Integral takes ownership of args + /* TODO: when we will approximate integral, we might want to simplify the * integral here. However, we might want to do it once for all x (to avoid * lagging in the derivative table. */ - return PoincareHelpers::ApproximateToScalar(&integral, *context); + return PoincareHelpers::ApproximateToScalar(integral, *context); } Expression::Coordinate2D CartesianFunction::nextMinimumFrom(double start, double step, double max, Context * context) const { - return expression(context)->nextMinimum(symbol(), start, step, max, *context, Preferences::sharedPreferences()->angleUnit()); + return expression(context).nextMinimum(symbol(), start, step, max, *context, Preferences::sharedPreferences()->angleUnit()); } Expression::Coordinate2D CartesianFunction::nextMaximumFrom(double start, double step, double max, Context * context) const { - return expression(context)->nextMaximum(symbol(), start, step, max, *context, Preferences::sharedPreferences()->angleUnit()); + return expression(context).nextMaximum(symbol(), start, step, max, *context, Preferences::sharedPreferences()->angleUnit()); } double CartesianFunction::nextRootFrom(double start, double step, double max, Context * context) const { - return expression(context)->nextRoot(symbol(), start, step, max, *context, Preferences::sharedPreferences()->angleUnit()); + return expression(context).nextRoot(symbol(), start, step, max, *context, Preferences::sharedPreferences()->angleUnit()); } Expression::Coordinate2D CartesianFunction::nextIntersectionFrom(double start, double step, double max, Poincare::Context * context, const Shared::Function * function) const { - return expression(context)->nextIntersection(symbol(), start, step, max, *context, Preferences::sharedPreferences()->angleUnit(), function->expression(context)); + return expression(context).nextIntersection(symbol(), start, step, max, *context, Preferences::sharedPreferences()->angleUnit(), function->expression(context)); } char CartesianFunction::symbol() const {