[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Fix comment for Box::unite (issue 559870043 by address@hidden)
From: |
hanwenn |
Subject: |
Re: Fix comment for Box::unite (issue 559870043 by address@hidden) |
Date: |
Sat, 18 Apr 2020 02:32:15 -0700 |
Reviewers: lemzwerg, Dan Eble,
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#newcode38
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
Description:
Fix comment for Box::unite
Please review this at https://codereview.appspot.com/559870043/
Affected files (+2, -1 lines):
M lily/include/box.hh
Index: lily/include/box.hh
diff --git a/lily/include/box.hh b/lily/include/box.hh
index
6eb1a681d2c38da0290cba4f3a9a06be5dffdda2..5cd5b002336759e5818c607f5de8dcc8adca51e0
100644
--- a/lily/include/box.hh
+++ b/lily/include/box.hh
@@ -30,11 +30,12 @@ public:
void translate (Offset o);
- /// smallest box enclosing #b#
void set_empty ();
void add_point (Offset);
void widen (Real x, Real y);
void scale (Real r);
+
+ /// smallest box enclosing `this` and `b`
void unite (Box b);
void intersect (Box b);
void print ();