atexit: Clarify DLL behavior is the same as _onexit - #5952
atexit: Clarify DLL behavior is the same as _onexit#5952Jay Satiro (jay) wants to merge 2 commits into
Conversation
- Document that atexit called from within a DLL will register the routine to run when the DLL is unloaded. atexit is a wrapper around _onexit which already documents this behavior. Ref: curl/curl#22383 (comment) Reported-by: Michał Petryka Closes #xxxx
|
Jay Satiro (@jay) : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change. |
1 similar comment
|
Jay Satiro (@jay) : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change. |
|
_crt_atexit may also need this information? I'm not sure since I do not have the source for that. |
|
Learn Build status updates of commit 4b73d28: ✅ Validation status: passed
For more details, please refer to the build report. |
|
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, adding a #label:"aq-pr-triaged" |
Update the date and clarify DLL behavior for atexit.
|
Thank you! |
Tyler Whitney (TylerMSFT)
left a comment
There was a problem hiding this comment.
Thank you!
No worries about crtatexit(). Its behavior is different.
|
Learn Build status updates of commit 1876cd3: ✅ Validation status: passed
For more details, please refer to the build report. |
|
#sign-off |
atexit is a wrapper around _onexit which already documents this behavior.
Ref: curl/curl#22383 (comment)
Reported-by: Michał Petryka
Closes #xxxx
I looked in the CRT source for 2010 (pre-Universal CRT) and 2022 (Universal CRT) and in both atexit is a wrapper around _onexit.
C:\Program Files (x86)\Microsoft Visual Studio 10.0\VC\crt\src\onexit.c
C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.31.31103\crt\src\vcruntime\utility.cpp
/cc Michał Petryka (@MichalPetryka)