STYLE: Remove unused jacjjacj variable from ComputeUsingSearchDirection

SuperElastix / elastix

Official elastix repository

http://elastix.dev

Apache License 2.0

449 stars 110 forks source link

STYLE: Remove unused jacjjacj variable from ComputeUsingSearchDirection #1142

Closed N-Dekker closed 2 weeks ago

N-Dekker commented 3 weeks ago

The unused local jacjjacj variable appears to be introduced with commit e9de4b1281a07d2fdbaa96db865f3c7aaaf4e5d2 "ENH: manual syncing with the performance branch", 25 May 2016.

In other cases, such a variable is (post-)initialized by vnl_fastops::ABt(jacjjacj, jacj, jacj), but in this case, it was left uninitialized.

N-Dekker commented 2 weeks ago

@mstaring FYI, I'll just merge this PR now, but if we ever want to have this unused variable (jacjjacj) back here (and actually use it), it's easy to put it back any time later.