[PATCH] Cygwin: More profiler format + small issue fixes

Corinna Vinschen corinna-cygwin@cygwin.com
Tue Aug 3 08:07:17 GMT 2021


On Aug  1 23:52, Mark Geisert wrote:
> Make sure to cast to ulong all DWORD values displayed with format "%lu".
> More instances are fixed here than in either my earlier unused patch or
> Corinna's patch. I decided to use typedef..ulong for more compact code.
> 
> Address jturney's reported small issues:
> - Remove explicit external ref for cygwin_internal() as it is already
>   provided by <sys/cygwin.h>.
> - Leave intact ref for cygwin_dll_path[] as it is required by function(s)
>   in path.cc that profiler uses. Added comment to that effect.
> - Delete existing main() wrapper. Rename main2() to main(). This because
>   profiler is now a Cygwin program and doesn't need to dynamically load
>   cygwin1.dll.
> - Documentation issues will be addressed in a separate xml patch.
> 
> (I would have linked message-ids of Corinna's and Jon's messages for
> proper theading but I no longer have their original emails and the mail
> archives don't show msgids any more.)
> 
> ---
>  winsup/utils/profiler.cc | 60 ++++++++++++++--------------------------
>  1 file changed, 20 insertions(+), 40 deletions(-)

Pushed.


Thanks,
Corinna


More information about the Cygwin-patches mailing list