[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    Re: kdesupport/taglib
From:       Scott Wheeler <wheeler () kde ! org>
Date:       2010-04-12 18:45:50
Message-ID: F3127095-BBCA-42A8-BD86-0ED2E294A41C () kde ! org
[Download RAW message or body]


On Apr 12, 2010, at 8:27 PM, Lukáš Lalinský wrote:
>
> int String::toInt() const
> {
> +  return toInt(0);
> +}
> +
> +int String::toInt(bool *ok) const

You might consider adding a // BIC: combine these two there.   
Previously I'd been adding similar notes so that it's easy to grep /  
fix when there's someday a binary incompatible release.

-Scott
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic