SDR should not auto-cast if vector provided instead of SDR as argument

[breznak]

@ctrl-z-9000-times You've mentioned the bug, and I ran into it in #397 with cellsToColumns

Problem: compiler(?) converts a wrongly-passed type vector to a SDR if SDR is a method's argument.

Example:

void bla(const SDR& input);

SDR ok;
vector<UInt> shouldFail;

bla(ok); //ok
bla(shouldFail); // should fail, as incompatible type provided, but instead this gets converted to a new SDR, which is totally wrong, as uses the data, instead of dimensions. 

Hint: I'd suspect some of SDR's default constructors, or assign operators

[breznak]

see 127e385

[ctrl-z-9000-times]

I don't think we're going to be able to stop the compiler from doing this.

We can detect this issue by checking the SDR's dimensions. Regardless of this bug, we should check SDR dimensions on most public facing API methods.

[breznak]

We can detect this issue by checking the SDR's dimensions. Regardless of this bug, we should check SDR dimensions on most public facing API methods.

would be nice if we could stop that behavior, if not, this should be documented somewhere (in SDR?) along with the advice to check SDR.dimensions