Skip to content

Conversation

@arrowd
Copy link
Contributor

@arrowd arrowd commented Aug 11, 2025

See commits descriptions.

arrowd added 2 commits August 11, 2025 11:59
This fixes a const char* -> void* invalid cast error with recent Clang:

src/shared/debug/dbgutil/elfreader.cpp:665:33: error: cannot initialize a parameter of type 'void *' with an rvalue of type 'const char *'
  665 |                 if (!ReadMemory(map.l_name + i, &ch, sizeof(ch))) {
      |                                 ^~~~~~~~~~~~~~
/home/arr/projects/dotnet/diagnostics/src/shared/debug/dbgutil/elfreader.h:78:35: note: passing argument to parameter 'address' here
   78 |     virtual bool ReadMemory(void* address, void* buffer, size_t size) = 0;
      |                                   ^
@arrowd arrowd requested a review from a team as a code owner August 11, 2025 09:01
Copy link
Member

@noahfalk noahfalk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks @arrowd!

@noahfalk noahfalk merged commit a321e26 into dotnet:main Aug 12, 2025
19 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Sep 12, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants