Improve readability around ForEach-function style in WorldPartition

Development Platform & Builds
, ,
1

Hi,

With the World Partition, a lot of new API calls take the form of a variation of ForEach(TFunction<>). Normally those TFunction<> (or templated parameter function) need to return true if they want to continue processing and false if they want to stop the processing (or was it the reverse?).

Would it be possible to change those API to instead use an enum, something similar to what is added at the end of this post.

This would increase readability and reduce possible misunderstanding around true/false returns.

Thank you for looking into it!

Have a nice day!

enum class EForEachState : uint8 { StopProcessing = 0, Continue }

2

Thanks for the context and the suggestion. I’ve passed this along to the rest of the team to consider.

Have a nice day, too!