[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: T1265 - Remove deprecation warnings when running with Guile V2 (issu
From: |
ianhulin44 |
Subject: |
Re: T1265 - Remove deprecation warnings when running with Guile V2 (issue2204044) |
Date: |
Mon, 18 Oct 2010 19:25:20 +0000 |
http://codereview.appspot.com/2204044/diff/14001/flower/include/guile-compatibility.hh
File flower/include/guile-compatibility.hh (right):
http://codereview.appspot.com/2204044/diff/14001/flower/include/guile-compatibility.hh#newcode28
flower/include/guile-compatibility.hh:28: #define scm_to_unsigned(x)
scm_to_uint (x)
On 2010/10/08 02:35:44, Patrick McCarty wrote:
This macro is not needed, either, AFAICS.
If you say
#define FOO 1
all instances of FOO are replaced with 1.
With this patch, you do away with the one instance of
"scm_to_unsigned(x)",
replacing it with "scm_to_uint(x)". Since "scm_to_unsigned(x)" no
longer
appears in the code base (after applying this patch), this macro does
nothing.
Done.
http://codereview.appspot.com/2204044/diff/14001/flower/include/guile-compatibility.hh#newcode30
flower/include/guile-compatibility.hh:30: #else // SCM_MINOR_VERSION >
6 && SCM_MINOR_VERSION >= 9
On 2010/10/08 02:35:44, Patrick McCarty wrote:
Just about :)
The comment is now redundant...
This should be
#else // SCM_MINOR_VERSION >= 9
Done.
http://codereview.appspot.com/2204044/diff/14001/lily/dispatcher.cc
File lily/dispatcher.cc (right):
http://codereview.appspot.com/2204044/diff/14001/lily/dispatcher.cc#newcode195
lily/dispatcher.cc:195: }
On 2010/10/08 02:35:44, Patrick McCarty wrote:
Please fix the indentation here (revert to using a tab).
Done.
http://codereview.appspot.com/2204044/diff/14001/lily/include/lily-guile-macros.hh
File lily/include/lily-guile-macros.hh (right):
http://codereview.appspot.com/2204044/diff/14001/lily/include/lily-guile-macros.hh#newcode146
lily/include/lily-guile-macros.hh:146: TYPE ## _ ## FUNC ##
_init_functions);
On 2010/10/08 02:35:44, Patrick McCarty wrote:
Revert back to tabs here.
Done.
http://codereview.appspot.com/2204044/