Subject: [ggl] [quickbook] Algorithms quickbook sample
From: Mateusz Loskot (mateusz)
Date: 2010-02-16 19:19:42
Barend Gehrels wrote:
> Hi Mateusz,
> Herewith some feedback on the compilation.
> Great it is there and it is looking very good, again.
> Some remarks:
> - depdendencies typos
> - " properly by following instructions..." actually the only thing
> necessary is to set the include folder, which is indeed written there,
> so we can lower this a bit probably
I've tried to update, see if it's OK now.
> - compilers, there are more now. It is also compiling on CodeWarrior (at
> least the intersections). What is our strategy here? It probably must
> run all unit tests. OK, then we cannot mention that one. I tested MSVC
> 2010 express beta, but didn't do all unit tests either...
Hmm, good question. What about maintaining two categories/lists:
1. Compilers confirmed as building full set of tests and examples
2. Compilers users mentioned but without providing further details
what extent of BG was built with it
> - Boost Geometry uses Boost Build, I would mention here that this is for
> unit tests. For users, it is really not necessary to use Boost Build
I believe it says so:
Boost Geometry uses Boost Build, ..., to configure, build and execute
unit tests and example programs.
> - Extensions: indeed, let's move that to another node, I like the pages
> to be a bit concise.
May be even move Extensions after Misc, so the top nodes would be draw:
> - However, I would include the "Intellisense issues" from the original
> page, because that bug really can be a irritating, reviewers mentioned
> this and probably were glad by this hint
I believe it is included:
Yes, I agree this bug is irritating. I've been suffering from it myself.
> - The advanced includes might maybe have better overview in a table? I
> know it was not like that in the original, but it just occurred to me
> now that it might look better.
Sure, good idea.
So, you mean table with one header per row and two columns? This way?
[ header | description ]
[ .../*.hpp | ... ]
-- Mateusz Loskot http://mateusz.loskot.net
Geometry list run by mateusz at loskot.net