On 2020/04/13 19:08:31, hanwenn wrote: > https://codereview.appspot.com/576000043/diff/577750046/flower/include/offset.hh > File flower/include/offset.hh (right): > > https://codereview.appspot.com/576000043/diff/577750046/flower/include/offset.hh#newcode123 > flower/include/offset.hh:123: Offset normal() const { > On 2020/04/13 18:24:47, dak wrote: > > It's kind of unusual to make a constructing function a (non-static) member > > function unless this is necessitated for operator semantics, because different > > conversion rules apply and there is some expectation that member functions > > delivering the class type may change stuff in-place (the const is not seen at > > call sites). > > > > Any reason to depart from conventions here? > > It's consistent with offset::direction, offset::swapped, but I followed your > suggestion.
You are right regarding direction and swapped, so your proposed change was not creating precedent and with the reason of maintaining convention/consistency untenable, I think I prefer your originally proposed syntax. Sorry for the noise. Any input from our C++ gurus? https://codereview.appspot.com/576000043/