Make [SD]rotm{,g} more idiomatic Go

We now have an int Flags type that indicates the status of the H matrix
and the [SD]RotmParams struct is returned as a concrete type since it is
not intended to be mutated by the rotm routine.

This also picked up a non-allocation bug in goblas.Drotmg.