Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: README.md

Issue 5163715573841920: Issue 768 - Switch from TR1 to C++11 (Closed)
Patch Set: rebase Created Aug. 5, 2015, 10:38 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: README.md
diff --git a/README.md b/README.md
index d72ed650ce8cf1465199345bd01d9c752e8a2543..87b71ac8e43011627338c79fbd48afe6ae402a59 100644
--- a/README.md
+++ b/README.md
@@ -51,6 +51,22 @@ Visual Studio and build the solution there. Alternatively you can use the
`msbuild` command line tool, e.g. run `msbuild /m build\ia32\libadblockplus.sln`
from the Visual Studio Developer Command Prompt to create a 32 bit debug build.
+### Supported compilers
+
+You need a C\+\+11 compatible compiler to build libadblockplus. If you need a C++98 compatible interface or you have a questions about another comiler please [create an
Wladimir Palant 2015/08/05 11:19:39 Nit: Please break that line at 80 characters. Nit
sergei 2015/08/06 07:35:25 Thanks.
+issue](https://issues.adblockplus.org/).
Wladimir Palant 2015/08/05 11:19:39 Will we actually create a C++98 compatible interfa
Felix Dahlke 2015/08/05 21:28:04 Fair enough, probably not. I suppose anyone who re
sergei 2015/08/06 07:35:25 Removed mentioning of C++98.
+
+The following C++11 features are currently used:
+* std::shared_ptr
+* std::function
Wladimir Palant 2015/08/05 11:19:39 IMHO, this list is bound to get out of date. Yes,
Felix Dahlke 2015/08/05 21:28:03 I was about to object but, rethinking it, I suppos
sergei 2015/08/06 07:35:26 Agree that there is a big chance to get it out of
+
+The compilation was tested with the following compilers:
+* g++ 4.8
+* Microsoft Visual Studio 2010, 2012
+* clang 3.4 for Android (from android-ndk-r9)
+* clang 3.6 for OS X
+
Felix Dahlke 2015/08/05 21:28:03 Nit: Just one new line between sections, as elsewh
+
Usage
-----
« no previous file with comments | « Makefile ('k') | common.gypi » ('j') | include/AdblockPlus/DefaultFileSystem.h » ('J')

Powered by Google App Engine
This is Rietveld