Hedos wrote:
It is quite simple and definitly not concerning Wine's core code, but I thought it'd be a good start.
Hi Philippe,
I had a quick look at your patch, and here are my comments:
+ if( !(newString = (TCHAR*)HeapAlloc(GetProcessHeap(), 0, lenbytes)) ) + return NULL; + memset((LPBYTE)newString, 0, lenbytes); + int i = 0, charcount = 0, newlinepos = 0;
There's no need to cast HeapAlloc returns or memset arguments.
You shouldn't declare "int i..." inline, as that isn't compatible with some compilers that Wine aims to support.
+ if(newLine) { //If last iteration was also a new line, this is an empty new line
We don't use C++ style comments for better compiler compatibility too.
+ HeapFree(GetProcessHeap(), 0, newString); + if (lRet == ERROR_SUCCESS) result = TRUE; + else error_code_messagebox(hwnd, lRet); + } else HeapFree(GetProcessHeap(), 0, newString);
The HeapFree() calls are common to both code pathes here, why not just do a single HeapFree() call?
+++ programs/regedit/main.c 24 Apr 2006 06:41:43 -0000 @@ -182,6 +182,8 @@ int APIENTRY WinMain(HINSTANCE hInstance } hAccel = LoadAccelerators(hInstance, (LPCTSTR)IDC_REGEDIT);
+ SetWindowText(hFrameWnd, "test v1.0");
Perhaps you meant to remove the above line before sending the patch?
Please try copy the coding style (curly brace placement, indentation and comments style) of the file you're changing, so we don't end up with a mismash of coding styles in each file.
Mike