tree fd235f1825a41901e9a3fafe3fd3a0825b52a141
parent 4cd257cf4ac8b0b81ab76b5dbac53b2b01de885e
author Sameer Agarwal <sameeragarwal@google.com> 1663593918 -0700
committer Sameer Agarwal <sameeragarwal@google.com> 1663599207 -0700

Fix a bug in TrustRegionMinimizer.

The value of x_norm_ was computed and then incorrectly set to -1.
This meant that ParameterToleranceReached was using the incorrect
value till such time as the minimizer made its first successful
step.

This change removes the member variable and just computes
the norm of x inside ParameterToleranceReached.

Fixes https://github.com/ceres-solver/ceres-solver/issues/885

Thanks to Anton Adanasyev for reporting this.

Change-Id: Ib4d52a45c2d925557ce2c5b57de8a9fa37da6c70