Windows specific changes. All tests passing on Windows.

include/AdblockPlus/FilterEngine.h

Index: include/AdblockPlus/FilterEngine.h
===================================================================
--- a/include/AdblockPlus/FilterEngine.h
+++ b/include/AdblockPlus/FilterEngine.h
@@ -4,7 +4,13 @@
 #include <vector>
 #include <map>
 #include <string>
+#ifdef _WIN32 || _WIN64

[Wladimir Palant, 2013/04/09 05:47:07]

I would prefer #if _MSC_VER_ here - it's cleaner, just in case we ever try
compiling via gcc on Windows.

+#include <memory>
+#include <cctype>
+#include <functional>

[Wladimir Palant, 2013/04/09 05:47:07]

Are these two headers really required on Windows? What for?

[Oleksandr, 2013/04/09 05:54:21]

<cctype> - for isspace
<functional> - for not1 and ptr_fun

[Wladimir Palant, 2013/04/09 06:43:48]

Then they should be added at the top of FilterEngine.cpp, right after algorithm.
No need for it to be Windows-only either, sounds like they really should be
there on Linux/Android as well.

[Felix Dahlke, 2013/04/09 08:18:06]

If it compiles on Linux/Android without them, they shouldn't be there IMO.
Unused includes make things pretty complex over time.

[Wladimir Palant, 2013/04/09 08:38:32]

We are using functions from these headers. It merely compiles because these
headers are included somewhere, probably in algorithm.h. So they are by no means
unused, if we omit them we expose ourselves to issues due to internal compiler
changes.

+#else
 #include <tr1/memory>
+#endif
 
 namespace AdblockPlus
 {