LGTM https://codereview.appspot.com/559870043/diff/553910043/lily/include/box.hh File lily/include/box.hh (right): https://codereview.appspot.com/559870043/diff/553910043/lily/include/box.hh#n... lily/include/box.hh:38: /// smallest box enclosing `this` and `b` Why '///' instead of '//'?
On 2020/04/17 17:17:27, lemzwerg wrote: > https://codereview.appspot.com/559870043/diff/553910043/lily/include/box.hh#n... > lily/include/box.hh:38: /// smallest box enclosing `this` and `b` > Why '///' instead of '//'? It's a habit some people have picked up from various experiences: https://stackoverflow.com/questions/9187616/three-forward-slashes-for-block-c...
https://codereview.appspot.com/559870043/diff/553910043/lily/include/box.hh File lily/include/box.hh (right): https://codereview.appspot.com/559870043/diff/553910043/lily/include/box.hh#n... lily/include/box.hh:38: /// smallest box enclosing `this` and `b` On 2020/04/17 17:17:27, lemzwerg wrote: > Why '///' instead of '//'? I just copied the line. The /// comes from a far away time that we tried to use doxygen