[Local GC] Use standard C++ types (bool) and consistent types (void*) on the interface (#10463)

* [Local GC] BOOL -> bool on IGCHeap

* [Local GC] size_t -> void* on IGCHeap

* [Local GC] Silence warnings by being explicit about BOOl -> bool conversions

* Address code review feedback: FinalizeAppDomain BOOL -> bool

* Fix warnings

* Address code review feedback:
  1) Fix a missed default parameter (FALSE) on a parameter of type bool,
  2) Fix invocations of the diagnostic callbacks to use boolean literals
     instead of TRUE and FALSE,
  3) Fix various invocations of GC interface methods in the VM to use
     boolean literals instead of TRUE and FALSE

* Address code review feedback: fix inconsistency

1 files changed, 3 insertions, 3 deletions

diff --git a/src/vm/gcheaputilities.h b/src/vm/gcheaputilities.h
index d5ce46c60d..0680763e76 100644
--- a/src/vm/gcheaputilities.h
+++ b/src/vm/gcheaputilities.h
@@ -81,7 +81,7 @@ public:
 
     // Returns true if a the heap is initialized and a garbage collection
     // is in progress, false otherwise.
-    inline static BOOL IsGCInProgress(BOOL bConsiderGCStart = FALSE)
+    inline static bool IsGCInProgress(bool bConsiderGCStart = false)
     {
         WRAPPER_NO_CONTRACT;
 
@@ -90,7 +90,7 @@ public:
 
     // Returns true if we should be competing marking for statics. This
     // influences the behavior of `GCToEEInterface::GcScanRoots`.
-    inline static BOOL MarkShouldCompeteForStatics()
+    inline static bool MarkShouldCompeteForStatics()
     {
         WRAPPER_NO_CONTRACT;
 
@@ -98,7 +98,7 @@ public:
     }
 
     // Waits until a GC is complete, if the heap has been initialized.
-    inline static void WaitForGCCompletion(BOOL bConsiderGCStart = FALSE)
+    inline static void WaitForGCCompletion(bool bConsiderGCStart = false)
     {
         WRAPPER_NO_CONTRACT;