xtensor-stack / xtensor-fftw

FFTW bindings for the xtensor C++14 multi-dimensional array library
BSD 3-Clause "New" or "Revised" License
48 stars 16 forks source link

use auto as return type and optimize in _dimensions_from_output #40

Closed marty1885 closed 6 years ago

marty1885 commented 6 years ago

Some optimizations and clean up I came across when trying to implement #37

coveralls commented 6 years ago

Coverage Status

Coverage remained the same at 98.597% when pulling 968a36f204ed4328728705e1fd6f0c2961be7472 on marty1885:master into 90c513c1228002b7823a0b9bd42fdaaaceace0b1 on egpbos:master.

marty1885 commented 6 years ago

@egpbos May you spend a minute to review this PR?

egpbos commented 6 years ago

Looks great, I thought I had already merged it, thanks!