I noticed some comments, warning or saying how things work, that are outdated, wrong, or misleading.

boinc/db/boinc_db.h line 23
// Structures corresponding to database records.
// Some of these types have counterparts in [b]client/types.h[/b],
// but don't be deceived - client and server have different variants.

I didn't find such file, I guess it means client/client_types.h. There are also the same struct names in lib/gui_rpc_client.h (so that makes triplicated names...)

boinc/db/boinc_db.h line 332
double rsc_memory_bound;    // upper bound on RAM working set (bytes)
// currently used only by scheduler to screen hosts
// [b]At some point, could use as runtime limit[/b]

I believe it's already being used as runtime limit on recent BOINC versions (5.8.x).

I'll post more if I find (I'm not specifically looking for these problems, I just see them while looking for other things).

EDIT: why double line endings on and [pre]? That problem has been reported more than once...