Thanks for quick response!
First i see is that your mail application wrapped the patch ( that is when long likes brake down into the next line ). Then you have a whitespace issue in the makefile i think...
Looks like my mailer (Windows Live Mail) thinks that if attachment has .txt extension it is ok to do line wrapping and changing EOLs to CR LF. I couldn't find an option to turn this feature off, but sending patches with .patch extensions seems to prevent this formatting.
I created a new signal.c file for the test case under ntdll/tests since I couldn’t find more fitting place.
Not a good idea. Please put the test into rtl.c
Ok, I will do that.
Please put ok() check inside get_back_trace_with_invalid_frame_pointer().
Hmm, good point, I wonder why I didn't check the result -variable instead... I will change it as you proposed.
Please use HeapAlloc/HeapFree. Don't cast void pointers. Here you don't need to allocate anything, just use a stack variable.
True, using stack is easier, I don't know why I tried to do it the hard way.
Also you not checking the return value from RtlCaptureStackBackTrace(). Please do.
The function returns number of frames it could get. I don't know what kind of value should be checked here since RtlCaptureStackBackTrace() is implemented only on i386 architecture. The test would fail on other architectures like arm, powerpc, x86_64 and sparc.
Also, please send the test patch by itself through winebot before posting and verify that it passes on all flavors of windows.
Ok, I will try it out when I have fixed the test case.
Thanks, Janne Hakonen