Review Request: Fixes to allow kdevplatform to run on Windows

[Andreas Pakulat]

> On 2009-08-11 22:53:09, Andreas Pakulat wrote:
> > svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/language/duchain/repositories/typerepository.cpp, line 85
> > <http://reviewboard.kde.org/r/1273/diff/1/?file=9914#file9914line85>
> >
> >     You need to update your patch, this has been fixed differently already in trunk.
> 
>  wrote:
>     I just updated from trunk and I can't see any change. As a matter of fact, I still get the same crash I reported that this change is meant to fix.

Hmm, I must've mixed something up. Yeap this does look ok.


- Andreas


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1273/#review1984
-----------------------------------------------------------


On 2009-08-16 15:03:10, Yaron Hirsch wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/1273/
> -----------------------------------------------------------
> 
> (Updated 2009-08-16 15:03:10)
> 
> 
> Review request for KDevelop.
> 
> 
> Summary
> -------
> 
> Fixes to allow kdevplatform to run on Windows
> Changes to remove warnings in MSVC regarding types first defined as class/struct but declared as class/struct.
> There's another review for kdevelop regarding this bug.
> 
> 
> This addresses bug 201455.
>     https://bugs.kde.org/show_bug.cgi?id=201455
> 
> 
> Diffs
> -----
> 
>   svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/CMakeLists.txt 1011970 
>   svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/language/backgroundparser/parsejob.cpp 1011970 
>   svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/language/codecompletion/abstractincludefilecompletionitem.h 1011970 
>   svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/language/codecompletion/codecompletionitem.h 1011970 
>   svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/language/codecompletion/codecompletionmodel.cpp 1011970 
>   svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/language/codegen/applychangeswidget.cpp 1011970 
>   svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/language/codegen/codegenerator.cpp 1011970 
>   svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/language/codegen/createclass.cpp 1011970 
>   svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/language/codegen/documentchangeset.h 1011970 
>   svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/language/duchain/appendedlist.h 1011970 
>   svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/language/duchain/codemodel.cpp 1011970 
>   svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/language/duchain/definitions.cpp 1011970 
>   svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/language/duchain/duchainbase.h 1011970 
>   svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/language/duchain/ducontext.h 1011970 
>   svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/language/duchain/dumpdotgraph.cpp 1011970 
>   svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/language/duchain/functiondefinition.h 1011970 
>   svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/language/duchain/identifier.h 1011970 
>   svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/language/duchain/importers.cpp 1011970 
>   svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/language/duchain/instantiationinformation.h 1011970 
>   svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/language/duchain/persistentsymboltable.cpp 1011970 
>   svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/language/duchain/types/typesystemdata.h 1011970 
>   svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/language/duchain/use.h 1011970 
>   svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/language/duchain/uses.cpp 1011970 
>   svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/language/editor/modificationrevisionset.cpp 1011970 
>   svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/language/editor/simplecursor.h 1011970 
>   svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/language/editor/simplerange.h 1011970 
>   svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/language/util/basicsetrepository.h 1011970 
>   svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/language/util/includeitem.h 1011970 
>   svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/language/util/setrepository.cpp 1011970 
>   svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/plugins/genericprojectmanager/genericmanager.h 1011970 
>   svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/plugins/patchreview/libdiff2/diffexport.h 1011970 
>   svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/plugins/patchreview/libdiff2/kompare.h 1011970 
>   svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/plugins/patchreview/libdiff2/komparemodellist.h 1011970 
>   svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/plugins/patchreview/libdiff2/komparemodellist.cpp 1011970 
>   svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/plugins/patchreview/libdiff2/settingsbase.h 1011970 
>   svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/project/projectmodel.h 1011970 
>   svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/shell/partdocument.cpp 1011970 
>   svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/shell/textdocument.h 1011970 
>   svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/sublime/mainwindow.h 1011970 
>   svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/sublime/mainwindow_p.cpp 1011970 
> 
> Diff: http://reviewboard.kde.org/r/1273/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Yaron
> 
>