Здравствуйте, уважаемый Selavi!

Сложно сказать, что именно они считают как слабо.
Лично мне не понравилось, что члены классов никак не выделены:

class CThreadPoolX: public IFinishClientTask
{
...
bool terminated;
...
std::vector<WorkThreadPtr> threads;
std::thread thread;
};

в теле метода их можно запросто перепутать с локальной (автоматической) переменной.

Я бы дал названия членам класса — примерно так (венгерская нотация):

bool m_bTerminated;
std::vector<WorkThreadPtr> m_VectThreads;
std::thread m_thread;