Skip to content

Remove C-String PAL wrappers - #98189

Merged
jkoritzinsky merged 7 commits into
dotnet:mainfrom
jkoritzinsky:string-pal
Feb 10, 2024
Merged

jkoritzinsky merged 7 commits into
dotnet:mainfrom
jkoritzinsky:string-pal

Conversation

@jkoritzinsky

Copy link
Copy Markdown
Member

Remove PAL wrappers for strtoul, strtoull, and strnlen. Make the implementations of _stricmp, _strnicmp, and _strdup just forward to strcasecmp, strncasecmp, and strdup respectively.

@jkoritzinsky jkoritzinsky added the area-PAL-coreclr only for closed issues label Feb 8, 2024
@ghost ghost assigned jkoritzinsky Feb 8, 2024
@jkoritzinsky
jkoritzinsky requested a review from jkotas February 8, 2024 22:16
Comment thread src/coreclr/pal/inc/mbusafecrt.h
Comment thread src/coreclr/tools/StressLogAnalyzer/StressLogPlugin.cpp Outdated
Co-authored-by: Aaron Robinson <arobins@microsoft.com>
@jkoritzinsky
jkoritzinsky merged commit 96da5a0 into dotnet:main Feb 10, 2024
@jkoritzinsky
jkoritzinsky deleted the string-pal branch February 10, 2024 03:02
@github-actions github-actions Bot locked and limited conversation to collaborators Mar 12, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-PAL-coreclr only for closed issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants