Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(5199)

Issue 559870043: Fix comment for Box::unite (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
3 years, 11 months ago by hanwenn
Modified:
3 years, 10 months ago
Reviewers:
Dan Eble, lemzwerg
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Fix comment for Box::unite

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -1 line) Patch
M lily/include/box.hh View 1 chunk +2 lines, -1 line 2 comments Download

Messages

Total messages: 3
lemzwerg
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#newcode38 lily/include/box.hh:38: /// smallest box enclosing `this` and `b` Why ...
3 years, 11 months ago (2020-04-17 17:17:27 UTC) #1
Dan Eble
On 2020/04/17 17:17:27, lemzwerg wrote: > https://codereview.appspot.com/559870043/diff/553910043/lily/include/box.hh#newcode38 > lily/include/box.hh:38: /// smallest box enclosing `this` and ...
3 years, 11 months ago (2020-04-17 21:43:52 UTC) #2
hanwenn
3 years, 11 months ago (2020-04-18 09:32:16 UTC) #3
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
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b