[Kmymoney-devel] Review Request: Refactoring of matching a transaction-under-import

[Alvaro Soliverez]

BTW, when the Transaction objects are created, they are initialized, so I
really don't see why boost::optional should be used here.


On Sat, Dec 8, 2012 at 2:08 PM, Alvaro Soliverez <asoliverez at kde.org> wrote:

> I disagree. It's not about adding a non-Qt dependency, which KMyMoney
> already has a few. It's about adding a new dependency without a sound
> rationale, just for a convenience class that one or another developer has
> grown used to.
>
> I still don't see boost bringing in enough of an improvement to add a new
> dependency here.
>
> Besides this, my experience with boost has been horrible in the past.
> Removing or adding features for minor versions, API changing, no ABI
> compatibility and other stuff. It's not the kind of dependency I'd like to
> maintain.
>
> Regards,
> Alvaro
>
>
> On Sat, Dec 8, 2012 at 1:58 PM, Łukasz Maszczyński <lukasz at maszczynski.net
> > wrote:
>
>> needed for boost::optional though, and in my opinion we shouldn't get rid
>> of boost just because it's n
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kmymoney-devel/attachments/20121208/530781cc/attachment-0001.html>