lilypond-devel
[Top][All Lists]
Advanced

[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 ();





reply via email to

[Prev in Thread] Current Thread [Next in Thread]