wangkx/HPCC-Platform — comment on commit b2e222aa72b44feb8ce966a37cc76c1c0301919e
File:  line 0
By richardkchapman:

No, it wasn't isEmpty(). I don't think adding isEmpty() everywhere would do anything for the efficiency or readability of the code. What I am complaining about is code anywhere outside of the common dlls that embeds knowledge of the environment XML structure.

It looks like the code here is looking for the first target cluster that can be used to send a job to the named thor (no idea WHY you want that but...)

There is a function getEnvironmentThorClusterNames that might be of use... if there is NOTHING in workunit.cpp that gives you the info you want then you should create something.