Skip to content

Commit e59bbf7

Browse files
committed
Remove pAppDomainOverride parameter
1 parent fc87b8e commit e59bbf7

File tree

3 files changed

+7
-11
lines changed

3 files changed

+7
-11
lines changed

src/coreclr/debug/di/module.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,11 +65,11 @@ CordbModule::CordbModule(
6565
pProcess->GetDAC()->GetDomainAssemblyData(vmDomainAssembly, &dfInfo); // throws
6666

6767
m_pAppDomain = pProcess->LookupOrCreateAppDomain(dfInfo.vmAppDomain);
68+
_ASSERTE(m_pAppDomain == pProcess->GetAppDomain());
6869
m_pAssembly = m_pAppDomain->LookupOrCreateAssembly(dfInfo.vmDomainAssembly);
6970
}
7071
else
7172
{
72-
// Not yet implemented
7373
m_pAppDomain = pProcess->GetAppDomain();
7474
m_pAssembly = m_pAppDomain->LookupOrCreateAssembly(modInfo.vmAssembly);
7575
}

src/coreclr/debug/di/process.cpp

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -2294,10 +2294,10 @@ HRESULT CordbProcess::EnumerateHeapRegions(ICorDebugHeapSegmentEnum **ppRegions)
22942294

22952295
HRESULT CordbProcess::GetObject(CORDB_ADDRESS addr, ICorDebugObjectValue **ppObject)
22962296
{
2297-
return this->GetObjectInternal(addr, nullptr, ppObject);
2297+
return this->GetObjectInternal(addr, ppObject);
22982298
}
22992299

2300-
HRESULT CordbProcess::GetObjectInternal(CORDB_ADDRESS addr, CordbAppDomain* pAppDomainOverride, ICorDebugObjectValue **pObject)
2300+
HRESULT CordbProcess::GetObjectInternal(CORDB_ADDRESS addr, ICorDebugObjectValue **pObject)
23012301
{
23022302
HRESULT hr = S_OK;
23032303

@@ -2321,7 +2321,7 @@ HRESULT CordbProcess::GetObjectInternal(CORDB_ADDRESS addr, CordbAppDomain* pApp
23212321

23222322
CordbAppDomain *cdbAppDomain = NULL;
23232323
CordbType *pType = NULL;
2324-
hr = GetTypeForObject(addr, pAppDomainOverride, &pType, &cdbAppDomain);
2324+
hr = GetTypeForObject(addr, &pType, &cdbAppDomain);
23252325

23262326
if (SUCCEEDED(hr))
23272327
{
@@ -2557,7 +2557,7 @@ COM_METHOD CordbProcess::EnumerateLoaderHeapMemoryRegions(ICorDebugMemoryRangeEn
25572557
return hr;
25582558
}
25592559

2560-
HRESULT CordbProcess::GetTypeForObject(CORDB_ADDRESS addr, CordbAppDomain* pAppDomainOverride, CordbType **ppType, CordbAppDomain **pAppDomain)
2560+
HRESULT CordbProcess::GetTypeForObject(CORDB_ADDRESS addr, CordbType **ppType, CordbAppDomain **pAppDomain)
25612561
{
25622562
VMPTR_AppDomain appDomain;
25632563
VMPTR_Module mod;
@@ -2566,10 +2566,6 @@ HRESULT CordbProcess::GetTypeForObject(CORDB_ADDRESS addr, CordbAppDomain* pAppD
25662566
HRESULT hr = E_FAIL;
25672567
if (GetDAC()->GetAppDomainForObject(addr, &appDomain, &mod, &domainAssembly))
25682568
{
2569-
if (pAppDomainOverride)
2570-
{
2571-
appDomain = pAppDomainOverride->GetADToken();
2572-
}
25732569
CordbAppDomain *cdbAppDomain = appDomain.IsNull() ? GetAppDomain() : LookupOrCreateAppDomain(appDomain);
25742570

25752571
_ASSERTE(cdbAppDomain);

src/coreclr/debug/di/rspriv.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3699,7 +3699,7 @@ class CordbProcess :
36993699
// the jit attach.
37003700
HRESULT GetAttachStateFlags(CLR_DEBUGGING_PROCESS_FLAGS *pFlags);
37013701

3702-
HRESULT GetTypeForObject(CORDB_ADDRESS obj, CordbAppDomain* pAppDomainOverride, CordbType **ppType, CordbAppDomain **pAppDomain = NULL);
3702+
HRESULT GetTypeForObject(CORDB_ADDRESS obj, CordbType **ppType, CordbAppDomain **pAppDomain = NULL);
37033703

37043704
WriteableMetadataUpdateMode GetWriteableMetadataUpdateMode() { return m_writableMetadataUpdateMode; }
37053705
private:
@@ -4154,7 +4154,7 @@ class CordbProcess :
41544154
// controls how metadata updated in the target is handled
41554155
WriteableMetadataUpdateMode m_writableMetadataUpdateMode;
41564156

4157-
COM_METHOD GetObjectInternal(CORDB_ADDRESS addr, CordbAppDomain* pAppDomainOverride, ICorDebugObjectValue **pObject);
4157+
COM_METHOD GetObjectInternal(CORDB_ADDRESS addr, ICorDebugObjectValue **pObject);
41584158

41594159
#ifdef OUT_OF_PROCESS_SETTHREADCONTEXT
41604160
CUnmanagedThreadHashTableImpl m_unmanagedThreadHashTable;

0 commit comments

Comments
 (0)