Review Request: Fixes to allow kdevplatform to run on Windows

[Andreas Pakulat]

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



svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/language/codecompletion/abstractincludefilecompletionitem.h
<http://reviewboard.kde.org/r/1273/#comment1395>

    No need for a comment, just remove this.



svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/language/editor/modificationrevisionset.cpp
<http://reviewboard.kde.org/r/1273/#comment1396>

    You really should update this patch against current trunk. This code is completely different already.



svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/plugins/patchreview/libdiff2/diffexport.h
<http://reviewboard.kde.org/r/1273/#comment1397>

    This is uneeded by now too as there's no library being built anymore.



svn://anonsvn.kde.org/home/kde/trunk/KDE/kdevplatform/plugins/patchreview/libdiff2/settingsbase.h
<http://reviewboard.kde.org/r/1273/#comment1398>

    uneeded too.


- Andreas


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
> 
>