|
From: | lemzwerg |
Subject: | Re: Issue 4156: Define Smob<> constructors. (issue 152370043 by address@hidden) |
Date: | Wed, 08 Oct 2014 04:57:41 +0000 |
LGTM, from visual inspection only :-) https://codereview.appspot.com/152370043/diff/1/lily/include/book.hh File lily/include/book.hh (right): https://codereview.appspot.com/152370043/diff/1/lily/include/book.hh#newcode31 lily/include/book.hh:31: typedef Smob<Book> base_type; I would insert an empty line before the `public' keyword... https://codereview.appspot.com/152370043/
[Prev in Thread] | Current Thread | [Next in Thread] |