Code review feedback

src/coreclr/debug/daccess/dacdbiimpl.cpp

@@ -159,14 +159,14 @@ template<class T> void DeleteDbiMemory(T *p)
 // There's an inherent risk here - where each element's destructor will get called within
 // the context of the DAC. If the destructor tries to use the CRT allocator logic expecting
 // to hit the DBI's, we could be in trouble. Those objects need to use an export allocator like this.
-template<class T> void DeleteDbiArrayMemory(T *p, int len)
+template<class T> void DeleteDbiArrayMemory(T *p, int count)
 {
     if (p == NULL)
     {
         return;
     }
 
-    for (T *cur = p; cur < p + len; cur++)
+    for (T *cur = p; cur < p + count; cur++)
     {
         cur->~T();
     }

src/coreclr/debug/di/rspriv.h

@@ -168,6 +168,10 @@ struct MachineInfo
     USHORT m_usPort;
 };
 
+#ifdef DACCESS_COMPILE
+    #error This header cannot be used in the DAC
+#endif
+
 extern forDbiWorker forDbi;
 
 // for dbi we just default to new, but we need to have these defined for both dac and dbi

src/coreclr/debug/inc/dacdbiinterface.h

@@ -29,7 +29,7 @@
 //    In the DBI implementation, this can directly call delete (assuming the IAllocator::Free
 //    directly called new).
 template<class T> void DeleteDbiMemory(T *p);
-template<class T> void DeleteDbiArrayMemory(T *p, int len);
+template<class T> void DeleteDbiArrayMemory(T *p, int count);
 // Need a class to serve as a tag that we can use to overload New/Delete.
 class forDbiWorker {};
 extern forDbiWorker forDbi;