Refactor integral estimation and error handling

[alexandrehoffmann]

No description provided.

[Copilot]

Pull request overview

This PR refactors parts of the adaptive quadrature implementation by reusing local variables for sub-interval integral estimates and adjusting the remapped infinite-interval integrand evaluation.

Changes:

• Refactors adaptQuadrature() to reuse res / estimatedErr variables when splitting intervals.
• Removes NaN-masking logic in remapAndIntegrate() when computing the remapped integrand value.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Comments suppressed due to low confidence (1)

include/LNIT/AdaptiveQuadratures/AdaptiveQuadratureBase_impl.hpp:260

• remapAndIntegrate evaluates the mapped integrand at t = ±1 (CC nodes include ±1). At those endpoints t/(1-t*t) and dxdt divide by zero, so f(...) is called with ±∞ and fx*dxdt can become non-finite. The current isnan(fx*dxdt) check does not catch Inf, and after switching to integrate(fref, -1, 1) the local eps is unused (can trigger unused-variable warnings under -Werror).

Consider skipping endpoints and using isfinite on the mapped value (covers both NaN and Inf), and remove eps/isnan.

			? LongScalar{}
			: fx*dxdt;	
	};
	
	return integrate(fref, -1, 1);