Simplified the visit children guard to use a ValueGuardPtr. For consistency, so all three guards use the same pattern, I also updated the value guard, which is a side ways step in complexity (although less of it is under macros now). (details)
It seems clang uses different scoping rules for the trailing return of a scoped runction declaration. This form seems compatable with clang and gcc. Since I switched over to clang for testing I also cleaned up all errors that clang or gcc mentioned. (details)
Commit
4fc7388ea88eb10cdb80bc09e77bf2758b6beb01
by Peter Buhr
Commit
96c04e42a4fda498e09b123d4cb11501db114950
by ajbeach
Simplified the visit children guard to use a ValueGuardPtr. For consistency, so all three guards use the same pattern, I also updated the value guard, which is a side ways step in complexity (although less of it is under macros now).
Commit
822332e90fec6c4a9a05fd800863fb5009a81bcd
by ajbeach
It seems clang uses different scoping rules for the trailing return of a scoped runction declaration. This form seems compatable with clang and gcc. Since I switched over to clang for testing I also cleaned up all errors that clang or gcc mentioned.