On Thursday 14 June 2007, Allen Winter wrote:
> On Wednesday 13 June 2007 4:19:34 am David Faure wrote:
> > On Tuesday 12 June 2007, Allen Winter wrote:
> > > On Tuesday 12 June 2007 3:16:03 pm Christian Ehrlicher wrote:
> > > > SVN commit 674648 by chehrlic:
> > > > 
> > > > remove var name like discussed -> fix compile error
> > > > 
> > > I don't understand why this change is needed.
> > 
> > Because Q_UNUSED() requires the type to be fully defined with MSVC,
> > while gcc is happy to compile that line even when the type of the variable
> > is only a forward declaration. Yeah, MSVC sucks.
> > 
> So, how about we simply replace the 'class Item' with 'include <libakonadi/item.h>" ??
> Then we don't have the forward declaration to confuse MSVC.
> 
> I'd prefer that solution.

That solution raises the number of header inter-dependencies (and slightly increase compile time).
We forward-declare for a reason :-)

-- 
David Faure, faure@kde.org, dfaure@klaralvdalens-datakonsult.se
KDE/KOffice developer, Qt consultancy projects
Klarälvdalens Datakonsult AB, Platform-independent software solutions