Changeset 38013

Timestamp:

07/25/10 13:43:35 (3 years ago)

Author:

saturday06

Message:

sedai

Location:

lang/c/AutoUpdater/trunk/recipes

Files:

• AutoUpdater.vcproj (modified) (1 diff)
• Copy.c (modified) (2 diffs)
• Copy.vcproj (modified) (1 diff)
• Makefile.am (modified) (1 diff)
• Template.c (modified) (49 diffs)
• configure.ac (modified) (7 diffs)

lang/c/AutoUpdater/trunk/recipes/AutoUpdater.vcproj

@@ -199 +199 @@
                                 >
                         </File>
+                        <File
+                                RelativePath=".\XDef.h"
+                                >
+                        </File>
                 </Filter>
                 <Filter

lang/c/AutoUpdater/trunk/recipes/Copy.c

@@ +1 @@
+#ifndef _UNICODE
+#error
+#endif
+
 #include <windows.h>
-
-#define _T(x) TEXT(x)
+#include <tchar.h>
+#include <commctrl.h>
+
 
 /* http://msdn.microsoft.com/en-us/library/bb531404.aspx */
 /* swap _M_IX86 and _M_X64 */
-#ifdef UNICODE
 #if defined _M_X64
 #pragma comment(linker,"/manifestdependency:\"type='win32' name='Microsoft.Windows.Common-Controls' version='6.0.0.0' processorArchitecture='amd64' publicKeyToken='6595b64144ccf1df' language='*'\"")
@@ -14 +18 @@
 #else
 #pragma comment(linker,"/manifestdependency:\"type='win32' name='Microsoft.Windows.Common-Controls' version='6.0.0.0' processorArchitecture='*' publicKeyToken='6595b64144ccf1df' language='*'\"")
-#endif
 #endif
 

lang/c/AutoUpdater/trunk/recipes/Copy.vcproj

@@ -63 +63 @@
                         <Tool
                                 Name="VCLinkerTool"
+                                AdditionalDependencies="comctl32.lib"
                                 LinkIncremental="2"
                                 GenerateManifest="true"

lang/c/AutoUpdater/trunk/recipes/Makefile.am

@@ -3 +3 @@
 test_SOURCES   = Test.cpp Template.c
 test_CPPFLAGS  = -I.. -DSTRICT #-DUNICODE -D_UNICODE
+test_CFLAGS    = -O0 -g
+test_CXXFLAGS  = -O0 -g
 test_LDADD     = -lole32 -loleaut32 -luuid -lwininet -lshlwapi
 

lang/c/AutoUpdater/trunk/recipes/Template.c

@@ -79 +79 @@
     bytes = strlen(s) + 1;
     buf = (char*)calloc(bytes, 1);
-        if (!buf) {
+    if (!buf) {
         return NULL;
-        }
+    }
     memcpy(buf, s, bytes);
     return buf;
@@ -91 +91 @@
     bytes = (wcslen(s) + 1) * sizeof(wchar_t);
     buf = (wchar_t*)calloc(bytes, 1);
-        if (!buf) {
+    if (!buf) {
         return NULL;
-        }
+    }
     memcpy(buf, s, bytes);
     return buf;
@@ -103 +103 @@
     TCHAR* url;
     FILE* fp;
-
     HINTERNET internet_handle;
     HANDLE request_complete_event;
     int url_handle_requested;
     int wait_request_complete;
-    char buffer[1000];
-    INTERNET_BUFFERS ib;
-
+    char buffer[1024];
     HINTERNET url_handle;
     int complete;
+    INTERNET_BUFFERS* ib; /* pointer for -Wpadded */
 } AUI_DownloadContext;
 
@@ -151 +149 @@
 
 AU_Result E(AU_Error error) {
-        AU_Result result;
+    AU_Result result;
     result.success = 0;
     result.pending = 0;
-        result.error = error;
+    result.error = error;
     return result;
 }
@@ -170 +168 @@
     TCHAR* result = NULL;
     size_t result_len = 0;
-    
+
     result_len = _tcslen(base) + 1 + _tcslen(append) + 1;
     result = (TCHAR*)calloc(result_len * sizeof(TCHAR), 1);
@@ -206 +204 @@
 
     if (!WideCharToMultiByte(
-            CP_ACP,
-            0,
-            w,
-            -1,
-            m,
-            bytes,
-            NULL,
-            NULL
-            )) {
-        free(m);
-        return NULL;
+        CP_ACP,
+        0,
+        w,
+        -1,
+        m,
+        bytes,
+        NULL,
+        NULL
+        )) {
+            free(m);
+            return NULL;
     }
 
@@ -244 +242 @@
 
     if (!MultiByteToWideChar(
-            CP_ACP,
-            0,
-            m,
-            -1,
-            w,
-            count
-            )) {
-        free(w);
-        return NULL;
-    }
-    
+        CP_ACP,
+        0,
+        m,
+        -1,
+        w,
+        count
+        )) {
+            free(w);
+            return NULL;
+    }
+
     return w;
 }
@@ -286 +284 @@
 
     *result = NULL;
-    
+
     system_temp_path_chars = GetTempPath(0, NULL);
     system_temp_path = (TCHAR*)calloc(system_temp_path_chars * sizeof(TCHAR), 1);
@@ -302 +300 @@
         args[1] = (unsigned int)GetTickCount();
         if (!FormatMessage(
-                FORMAT_MESSAGE_ALLOCATE_BUFFER | FORMAT_MESSAGE_FROM_STRING | FORMAT_MESSAGE_ARGUMENT_ARRAY,
-                _T("AutoUpdater.%1!x!.%2!x!"),
-                0,
-                0,
-                (TCHAR*)&allocated,
-                0,
-                (va_list*)args)) {
-            return E(AU_ERROR_BAD_ALLOC);
+            FORMAT_MESSAGE_ALLOCATE_BUFFER | FORMAT_MESSAGE_FROM_STRING | FORMAT_MESSAGE_ARGUMENT_ARRAY,
+            _T("AutoUpdater.%1!x!.%2!x!"),
+            0,
+            0,
+            (TCHAR*)&allocated,
+            0,
+            (va_list*)args)) {
+                return E(AU_ERROR_BAD_ALLOC);
         }
         dir = (TCHAR*)allocated;
@@ -335 +333 @@
 /* http://support.microsoft.com/kb/238393 */
 HRESULT AUI_AutoWrap(WORD autoType, VARIANT *pvResult, IDispatch *pDisp,
-                 const OLECHAR* ptcName, size_t cArgs, VARIANT* rargs)
+                     const OLECHAR* ptcName, size_t cArgs, VARIANT* rargs)
 {
     DISPPARAMS dp = { NULL, NULL, 0, 0 };
@@ -354 +352 @@
     }
 
-        if (pvResult) {
-                VariantInit(pvResult);
-        }
+    if (pvResult) {
+        VariantInit(pvResult);
+    }
 
 #ifdef __cplusplus
@@ -389 +387 @@
 
 #ifdef __cplusplus
-    hr = pDisp->Invoke(dispID, IID_NULL, LOCALE_USER_DEFAULT, autoType,
-             &dp, pvResult, NULL, NULL);
+    hr = pDisp->Invoke(dispID, IID_NULL, LOCALE_USER_DEFAULT,
+                       autoType, &dp, pvResult, NULL, NULL);
 #else
-    hr = pDisp->lpVtbl->Invoke(pDisp, dispID, &IID_NULL, LOCALE_USER_DEFAULT, autoType,
-             &dp, pvResult, NULL, NULL);
+    hr = pDisp->lpVtbl->Invoke(pDisp, dispID, &IID_NULL, LOCALE_USER_DEFAULT, 
+                               autoType, &dp, pvResult, NULL, NULL);
 #endif
 
@@ -418 +416 @@
     }
 #ifdef __cplusplus
-        pd->AddRef();
+    pd->AddRef();
 #else
-        pd->lpVtbl->AddRef(pd);
+    pd->lpVtbl->AddRef(pd);
 #endif
 }
@@ -429 +427 @@
     }
 #ifdef __cplusplus
-        pd->Release();
+    pd->Release();
 #else
-        pd->lpVtbl->Release(pd);
+    pd->lpVtbl->Release(pd);
 #endif
 }
@@ -456 +454 @@
             return E(AU_ERROR_NETWORK);
         }
-                context->wait_request_complete = 1;
+        context->wait_request_complete = 1;
     }
 
@@ -473 +471 @@
     context->wait_request_complete = 0;
 
-        if (!context->url_handle) {
-                return E(AU_ERROR_SYSTEM);
-        }
+    if (!context->url_handle) {
+        return E(AU_ERROR_SYSTEM);
+    }
 
     {
@@ -483 +481 @@
         memset(&(context->buffer), 0, sizeof(context->buffer));
         memset(&(context->ib), 0, sizeof(context->ib));
-        context->ib.dwStructSize = sizeof(context->ib);
-        context->ib.lpvBuffer = context->buffer;
-        context->ib.dwBufferLength = sizeof(context->buffer);
+        context->ib->dwStructSize = sizeof(context->ib);
+        context->ib->lpvBuffer = context->buffer;
+        context->ib->dwBufferLength = sizeof(context->buffer);
         read_result = InternetReadFileEx(
-                          context->url_handle,
-                          &(context->ib),
-                          IRF_ASYNC | IRF_USE_CONTEXT | IRF_NO_WAIT,
-                          (DWORD_PTR)context);
+            context->url_handle,
+            context->ib,
+            IRF_ASYNC | IRF_USE_CONTEXT | IRF_NO_WAIT,
+            (DWORD_PTR)context);
         e = GetLastError();
         if (!read_result && e == ERROR_IO_PENDING) {
@@ -500 +498 @@
             return E(AU_ERROR_NETWORK);
         }
-        
-    }
-    if (context->ib.dwBufferLength == 0) {
+
+    }
+    if (context->ib->dwBufferLength == 0) {
         fclose(context->fp);
         context->fp = NULL;
@@ -509 +507 @@
     }
 
-    if (fwrite(context->buffer, 1, context->ib.dwBufferLength, context->fp) != context->ib.dwBufferLength) {
+    if (fwrite(context->buffer, 1, context->ib->dwBufferLength, context->fp) != context->ib->dwBufferLength) {
         return E(AU_ERROR_SYSTEM);
     }
@@ -515 +513 @@
 }
 
+/* http://msdn.microsoft.com/en-us/library/aa385121.aspx */
 void CALLBACK AUI_InternetStatusCallback(
     HINTERNET hInternet,
@@ -524 +523 @@
     AUI_DownloadContext* context = (AUI_DownloadContext*)dwContext;
 
+    (void)hInternet;
+    (void)dwStatusInformationLength;
+
     switch (dwInternetStatus) {
-    case INTERNET_STATUS_HANDLE_CREATED: 
+    case INTERNET_STATUS_CLOSING_CONNECTION:
+        /* Closing the connection to the server. The lpvStatusInformation 
+           parameter is NULL. */ 
+        {
+
+        }
+        break;
+    case INTERNET_STATUS_CONNECTED_TO_SERVER:
+        /* Successfully connected to the socket address (SOCKADDR) 
+           pointed to by lpvStatusInformation. */
+        {
+
+        }
+        break;
+    case INTERNET_STATUS_CONNECTING_TO_SERVER:
+        /* Connecting to the socket address (SOCKADDR) pointed to by 
+           lpvStatusInformation. */
+        {
+        }
+        break;
+    case INTERNET_STATUS_CONNECTION_CLOSED:
+        /* Successfully closed the connection to the server. The 
+           lpvStatusInformation parameter is NULL. */
+        {
+        }
+        break;
+#if 0
+    case INTERNET_STATUS_COOKIE_HISTORY:
+        /* Retrieving content from the cache. Contains data about past 
+           cookie events for the URL such as if cookies were accepted, 
+           rejected, downgraded, or leashed.
+
+           The lpvStatusInformation parameter is a pointer to an 
+           InternetCookieHistory structure. */
+        {
+        }
+        break;
+    case INTERNET_STATUS_COOKIE_RECEIVED:
+        /* Indicates the number of cookies that were accepted, rejected, 
+           downgraded (changed from persistent to session cookies), or 
+           leashed (will be sent out only in 1st party context). The 
+           lpvStatusInformation parameter is a DWORD with the number of 
+           cookies received. */
+        {
+        }
+        break;
+    case INTERNET_STATUS_COOKIE_SENT:
+        /* Indicates the number of cookies that were either sent or 
+           suppressed, when a request is sent. The lpvStatusInformation 
+           parameter is a DWORD with the number of cookies sent or 
+           suppressed. */
+        {
+        }
+        break;
+#endif
+    case INTERNET_STATUS_CTL_RESPONSE_RECEIVED:
+        /* Not implemented. */
+        {
+        }
+        break;
+#if 0
+    case INTERNET_STATUS_DETECTING_PROXY:
+        /* Notifies the client application that a proxy has been detected. */
+        {
+        }
+        break;
+#endif
+    case INTERNET_STATUS_HANDLE_CLOSING:
+        /* This handle value has been terminated. pvStatusInformation 
+           contains the address of the handle being closed. The 
+           lpvStatusInformation parameter contains the address of the 
+           handle being closed. */
+        {
+        }
+        break;
+    case INTERNET_STATUS_HANDLE_CREATED:
+        /* Used by InternetConnect to indicate it has created the new 
+           handle. This lets the application call InternetCloseHandle 
+           from another thread, if the connect is taking too long. The 
+           lpvStatusInformation parameter contains the address of an 
+           HINTERNET handle. */
         {
             INTERNET_ASYNC_RESULT* async_result = (INTERNET_ASYNC_RESULT*)lpvStatusInformation;
@@ -531 +613 @@
         }
         break;
-    case INTERNET_STATUS_CONNECTION_CLOSED:
+#if 0
+    case INTERNET_STATUS_INTERMEDIATE_RESPONSE:
+        /* Received an intermediate (100 level) status code message 
+           from the server. */
+        {
+        }
+        break;
+#endif
+    case INTERNET_STATUS_NAME_RESOLVED:
+        /* Successfully found the IP address of the name contained in 
+           lpvStatusInformation. The lpvStatusInformation parameter points 
+           to a PCTSTR containing the host name. */
+        {
+        }
+        break;
+#if 0
+    case INTERNET_STATUS_P3P_HEADER:
+        /* The response has a P3P header in it. */
+        {
+        }
+        break;
+    case INTERNET_STATUS_P3P_POLICYREF:
+        /* Not implemented. */
+        {
+        }
+        break;
+#endif
+    case INTERNET_STATUS_PREFETCH:
+        /* Not implemented. */
+        {
+        }
+        break;
+#if 0
+    case INTERNET_STATUS_PRIVACY_IMPACTED:
+        /* Not implemented. */
+        {
+        }
+        break;
+#endif
+    case INTERNET_STATUS_RECEIVING_RESPONSE:
+        /* Waiting for the server to respond to a request. The 
+           lpvStatusInformation parameter is NULL. */
+        {
+        }
+        break;
+    case INTERNET_STATUS_REDIRECT:
+        /* An HTTP request is about to automatically redirect the request. 
+           The lpvStatusInformation parameter points to the new URL. 
+           At this point, the application can read any data returned 
+           by the server with the redirect response and can query the 
+           response headers. It can also cancel the operation by closing 
+           the handle. This callback is not made if the original request 
+           specified INTERNET_FLAG_NO_AUTO_REDIRECT. */
+        {
+        }
         break;
     case INTERNET_STATUS_REQUEST_COMPLETE:
-        SetEvent(context->request_complete_event);
-        break;
+        /* An asynchronous operation has been completed. The 
+           lpvStatusInformation parameter contains the address of an 
+           INTERNET_ASYNC_RESULT structure. */
+        {
+            SetEvent(context->request_complete_event);
+        }
+        break;
+    case INTERNET_STATUS_REQUEST_SENT:
+        /* Successfully sent the information request to the server. The 
+           lpvStatusInformation parameter points to a DWORD value that 
+           contains the number of bytes sent. */
+        {
+        }
+        break;
+    case INTERNET_STATUS_RESOLVING_NAME:
+        /* Looking up the IP address of the name contained in 
+           lpvStatusInformation. The lpvStatusInformation parameter points 
+           to a PCTSTR containing the host name. */
+        {
+        }
+        break;
+    case INTERNET_STATUS_RESPONSE_RECEIVED:
+        /* Successfully received a response from the server. */
+        {
+        }
+        break;
+    case INTERNET_STATUS_SENDING_REQUEST:
+        /* Sending the information request to the server. The 
+           lpvStatusInformation parameter is NULL. */
+        {
+        }
+        break;
+#if 0
+    case INTERNET_STATUS_STATE_CHANGE:
+        /* Moved between a secure (HTTPS) and a nonsecure (HTTP) site. 
+           The user must be informed of this change; otherwise, the user 
+           is at risk of disclosing sensitive information involuntarily. 
+           When this flag is set, the lpvStatusInformation parameter points 
+           to a status DWORD that contains additional flags. */
+        {
+        }
+        break;
+#endif
     default:
         break;
@@ -550 +727 @@
     }
 
+    context->ib = (INTERNET_BUFFERS*)calloc(sizeof(*context->ib), 1);
+    if (!context->ib) {
+        return E(AU_ERROR_BAD_ALLOC);
+    }
+
     context->url = _tcsdup(url);
     if (!context->url) {
@@ -562 +744 @@
     _tfopen_s(&context->fp, context->file_path, _T("wb"));
 
-        if (!context->fp) {
-                return E(AU_ERROR_SYSTEM);
-        }
+    if (!context->fp) {
+        return E(AU_ERROR_SYSTEM);
+    }
 
     context->request_complete_event = CreateEvent(NULL, FALSE, FALSE, NULL);
-        if (!context->request_complete_event) {
-                return E(AU_ERROR_SYSTEM);
-        }
+    if (!context->request_complete_event) {
+        return E(AU_ERROR_SYSTEM);
+    }
 
     context->internet_handle = 
         InternetOpen(
-            TEXT("saturday06's AutoUpdater"), 
-            INTERNET_OPEN_TYPE_PRECONFIG, 
-            NULL, 
-            NULL, 
-            INTERNET_FLAG_ASYNC);
+        TEXT("saturday06's AutoUpdater"), 
+        INTERNET_OPEN_TYPE_PRECONFIG, 
+        NULL, 
+        NULL, 
+        INTERNET_FLAG_ASYNC);
 
     if (!context->internet_handle) {
@@ -584 +766 @@
 
     if (InternetSetStatusCallback(context->internet_handle, AUI_InternetStatusCallback)
-            == INTERNET_INVALID_STATUS_CALLBACK) {
-        return E(AU_ERROR_SYSTEM);
+        == INTERNET_INVALID_STATUS_CALLBACK) {
+            return E(AU_ERROR_SYSTEM);
     }
 
@@ -596 +778 @@
     if (AUI_IsBadReadPointer(context, sizeof(context)) || 
         AUI_IsBadWritePointer(context, sizeof(context))) {
-        return E(AU_ERROR_SYSTEM);
+            return E(AU_ERROR_SYSTEM);
+    }
+    if (context->ib) {
+        free(context->ib);
     }
     if (context->fp) {
@@ -623 +808 @@
 /*
 DWORD CALLBACK AUI_WindowsUpdateThread(LPVOID context_) {
-    AUI_UpdaterThread((AUI_Context*)context_);
-    return 0;
+AUI_UpdaterThread((AUI_Context*)context_);
+return 0;
 }
 */
@@ -634 +819 @@
         AUI_IsBadReadPointer(option, sizeof(*option)) ||
         AUI_IsBadReadPointer(option->url, 1)) {
-        return E(AU_ERROR_INVALID_OPTION);
+            return E(AU_ERROR_INVALID_OPTION);
     }
 
@@ -693 +878 @@
 void AUI_DeleteDirectory(const TCHAR* dir) {
     TCHAR* glob = NULL;
-        WIN32_FIND_DATA find_data;
+    WIN32_FIND_DATA find_data;
     HANDLE hFind = NULL;
 
@@ -701 +886 @@
 
     glob = AUI_PathAppendDup(dir, _T("*"));
-        hFind = FindFirstFile(glob, &find_data);
-        if( INVALID_HANDLE_VALUE == hFind ) {
-                return;
-        }
-
-        do {
+    hFind = FindFirstFile(glob, &find_data);
+    if( INVALID_HANDLE_VALUE == hFind ) {
+        return;
+    }
+
+    do {
         TCHAR* dir2 = NULL;
 
@@ -719 +904 @@
         dir2 = AUI_PathAppendDup(dir, find_data.cFileName);
 
-                if (FILE_ATTRIBUTE_DIRECTORY & find_data.dwFileAttributes) {
+        if (FILE_ATTRIBUTE_DIRECTORY & find_data.dwFileAttributes) {
             AUI_DeleteDirectory(dir2);
         } else {
             DeleteFile(dir2);
-                }
+        }
 
         free(dir2);
@@ -729 +914 @@
     } while(FindNextFile(hFind, &find_data));
 
-        FindClose(hFind);
+    FindClose(hFind);
     free(glob);
-        return;
+    return;
 }
 
@@ -788 +973 @@
     IDispatch* releaseNode = NULL;
     IDispatch* urlNode = NULL;
-    IDispatch* descriptionNode = NULL;
+    /* IDispatch* descriptionNode = NULL; */
 
     r = E(AU_ERROR_SYSTEM);
@@ -831 +1016 @@
             PVOID cast = 0;
             result = CoCreateInstance(&clsid, NULL, CLSCTX_SERVER, &IID_IDispatch, &cast);
-                        Microsoft_XMLDOM = (IDispatch*)cast;
+            Microsoft_XMLDOM = (IDispatch*)cast;
         }
 #endif
@@ -862 +1047 @@
     }
 
-    
+
     { /* Microsoft_XMLDOM.async = false */
         args[0].vt = VT_BOOL;
@@ -869 +1054 @@
     }
 
-    
+
     { /* Microsoft_XMLDOM.load("file.xml") */
         args[0].vt = VT_BSTR;
         /* args[0].bstrVal = SysAllocString(OLESTR("c:/Documents and Settings/saturday06/My Documents/My Dropbox/jsxml.xml")); */
-                /* args[0].bstrVal = SysAllocString(OLESTR("D:/Documents/My Dropbox/jsxml.xml")); */
+        /* args[0].bstrVal = SysAllocString(OLESTR("D:/Documents/My Dropbox/jsxml.xml")); */
         args[0].bstrVal = AUI_TCharToBStrDup(context->releases_file_path);
         result = AUI_AutoWrap(DISPATCH_METHOD, &returnValue, Microsoft_XMLDOM, OLESTR("load"), 1, args);
@@ -879 +1064 @@
             goto clean_up;
         }
-                if (!returnValue.boolVal) {
-                        goto clean_up;
-                }
+        if (!returnValue.boolVal) {
+            goto clean_up;
+        }
         VariantClear(&returnValue);
     }
@@ -888 +1073 @@
         args[0].vt = VT_BSTR;
         args[0].bstrVal = SysAllocString(OLESTR("SelectionLanguage"));
-                if (!args[0].bstrVal) {
-                        goto clean_up;
-                }
+        if (!args[0].bstrVal) {
+            goto clean_up;
+        }
         args[1].vt = VT_BSTR;
         args[1].bstrVal = SysAllocString(OLESTR("XPath"));
-                if (!args[1].bstrVal) {
-                        goto clean_up;
-                }
+        if (!args[1].bstrVal) {
+            goto clean_up;
+        }
         result = AUI_AutoWrap(DISPATCH_METHOD, NULL, Microsoft_XMLDOM, OLESTR("setProperty"), 2, args);
         if (FAILED(result)) {
@@ -901 +1086 @@
         }
     }
-    
+
     { /* var documentElement = Microsoft_XMLDOM.documentElement */
         result = AUI_AutoWrap(DISPATCH_PROPERTYGET, &returnValue, Microsoft_XMLDOM, OLESTR("documentElement"), 0, NULL);
@@ -919 +1104 @@
         memset(xpath, 0, sizeof(xpath));
         wsprintf(xpath, /* TODO */
-                 _T("/releases/release[not(@debug) or @debug <= %d][")
-                 _T("not(@revision <= preceding-sibling::release[not(@debug) or @debug <= %d]/@revision) and ")
-                 _T("not(@revision <= following-sibling::release[not(@debug) or @debug <= %d]/@revision)]")
+            _T("/releases/release[not(@debug) or @debug <= %d][")
+            _T("not(@revision <= preceding-sibling::release[not(@debug) or @debug <= %d]/@revision) and ")
+            _T("not(@revision <= following-sibling::release[not(@debug) or @debug <= %d]/@revision)]")
             , context->download_debug, context->download_debug, context->download_debug
             );
@@ -941 +1126 @@
         VariantClear(&returnValue);
     }
-    
+
     { /* var urlNode = releaseNode.selectSingleNode("url"); */
         args[0].vt = VT_BSTR;
@@ -984 +1169 @@
     r = AUI_SUCCESS;
 clean_up:
-        AUI_IDispatchRelease(urlNode);
-        AUI_IDispatchRelease(releaseNode);
+    AUI_IDispatchRelease(urlNode);
+    AUI_IDispatchRelease(releaseNode);
     AUI_IDispatchRelease(Microsoft_XMLDOM);
 
@@ -1074 +1259 @@
 
         _tfopen_s(&f, context->updater_config_file_path, _T("wb"));
-                if (!f) {
+        if (!f) {
             return E(AU_ERROR_SYSTEM);
-                }
+        }
 
         {
@@ -1104 +1289 @@
                 0,
                 (va_list*)args)) {
-                return E(AU_ERROR_BAD_ALLOC);
+                    return E(AU_ERROR_BAD_ALLOC);
             }
             data = AUI_TCharToWideCharDup((TCHAR*)allocated);
@@ -1117 +1302 @@
             }
         }
-                fclose(f);
+        fclose(f);
     }
 
@@ -1127 +1312 @@
         memset(&pi, 0, sizeof(pi));
         CreateProcess(context->updater_file_path, NULL, NULL, NULL, FALSE, 0, NULL, context->temp_dir_path, &si, &pi);
-            CloseHandle(pi.hThread);
-            CloseHandle(pi.hProcess);
+        CloseHandle(pi.hThread);
+        CloseHandle(pi.hProcess);
     }
 
@@ -1137 +1322 @@
 
 AU_Result AUI_Cancel(AUI_Context* context) {
+    (void)context;
     return AUI_SUCCESS;
 }
@@ -1143 +1329 @@
     AUI_Context* internal_context = (AUI_Context*)context;
     *internal_context_output = 0;
-        
-        if (AUI_IsBadReadPointer(internal_context, sizeof(internal_context))) {
-                return E(AU_ERROR_INVALID_CONTEXT);
-        }
-
-        if (AUI_IsBadWritePointer(internal_context, sizeof(internal_context))) {
-                return E(AU_ERROR_INVALID_CONTEXT);
-        }
-
-        if (internal_context->description != NULL && AUI_IsBadWritePointer(internal_context->description, 1)) {
-                return E(AU_ERROR_INVALID_CONTEXT);
-        }
-
-        if (internal_context->releases_file_path != NULL && AUI_IsBadWritePointer(internal_context->releases_file_path, 1)) {
-                return E(AU_ERROR_INVALID_CONTEXT);
-        }
-
-        if (internal_context->created_signature != AUI_CONTEXT_CREATED_SIGNATURE) {
-                return E(AU_ERROR_INVALID_CONTEXT);
-        }
+
+    if (AUI_IsBadReadPointer(internal_context, sizeof(internal_context))) {
+        return E(AU_ERROR_INVALID_CONTEXT);
+    }
+
+    if (AUI_IsBadWritePointer(internal_context, sizeof(internal_context))) {
+        return E(AU_ERROR_INVALID_CONTEXT);
+    }
+
+    if (internal_context->description != NULL && AUI_IsBadWritePointer(internal_context->description, 1)) {
+        return E(AU_ERROR_INVALID_CONTEXT);
+    }
+
+    if (internal_context->releases_file_path != NULL && AUI_IsBadWritePointer(internal_context->releases_file_path, 1)) {
+        return E(AU_ERROR_INVALID_CONTEXT);
+    }
+
+    if (internal_context->created_signature != AUI_CONTEXT_CREATED_SIGNATURE) {
+        return E(AU_ERROR_INVALID_CONTEXT);
+    }
 
     *internal_context_output = internal_context;
@@ -1169 +1355 @@
 
 /* -----------------------------------------------------------------------
-    Interfaces
-   ----------------------------------------------------------------------- */
+Interfaces
+----------------------------------------------------------------------- */
 
 #define AUI_API_PRECONDITION                       \
-        AUI_Context* internal_context = NULL;          \
-    {                                              \
-            AU_Result convert_result = AUI_ContextToInternalContext(&internal_context, context);  \
-            if (convert_result.error) {                \
-                return convert_result;                 \
-            }                                          \
-    }
+    AUI_Context* internal_context = NULL;          \
+{                                              \
+    AU_Result convert_result = AUI_ContextToInternalContext(&internal_context, context);  \
+    if (convert_result.error) {                \
+    return convert_result;                 \
+    }                                          \
+}
 
 AU_Result AU_CreateContext(AU_Context* pcontext, const AU_Option* option) {
     AU_Context context = NULL;
-        
+
     if (AUI_IsBadWritePointer(pcontext, sizeof(*pcontext))) {
         return E(AU_ERROR_INVALID_CONTEXT);

lang/c/AutoUpdater/trunk/recipes/configure.ac

@@ -8 +8 @@
 AC_CONFIG_SRCDIR([XDef.h])
 
-CFLAGS="-O0 -g"
-CXXFLAGS="-O0 -g"
+CFLAGS="-O0"
+CXXFLAGS="-O0"
 
 AC_PROG_CC
@@ -39 +39 @@
 
 cat >common_options.txt <<OPTIONS
-  -g
   -ansi
-  -pedantic
   -pedantic
   -Wextra  -Wall  -Waddress  -Warray-bounds 
@@ -49 +47 @@
   -Wdisabled-optimization  
   -Wempty-body  -Wenum-compare
-  -Wfloat-equal  -Wformat  -Wformat=2 
+  -Wfloat-equal -Wformat=2 
   -Wformat-nonliteral 
   -Wformat-security  -Wformat-y2k 
@@ -66 +64 @@
   -Wreturn-type  -Wsequence-point  -Wshadow 
   -Wsign-compare  -Wsign-conversion  -Wstack-protector 
-  -Wstrict-aliasing -Wstrict-aliasing=2
-  -Wstrict-overflow -Wstrict-overflow=2
+  -Wstrict-aliasing=2
+  -Wstrict-overflow=2
   -Wswitch  -Wswitch-default  -Wswitch-enum -Wsync-nand 
   -Wtrigraphs  -Wtype-limits  -Wundef  -Wuninitialized 
@@ -77 +75 @@
 OPTIONS
 # -Wsystem-headers 
-# -Waggregate-return 
+# -Waggregate-return <---- TODO
 
 cat >c_options.txt <<OPTIONS
@@ -88 +86 @@
   -Wstrict-prototypes 
   -Wdeclaration-after-statement -Wpointer-sign
+  -Wold-style-declaration  -Wold-style-definition 
 OPTIONS
+# -Wmissing-prototypes  <---- TODO
 # -Wmissing-declarations <---- TODO
-#-Wmissing-prototypes 
-#-Wtraditional  -Wtraditional-conversion 
-#-Wold-style-declaration  -Wold-style-definition 
+# -Wtraditional-conversion
+# -Wtraditional  
 
 cat >cxx_options.txt <<OPTIONS
@@ -111 +110 @@
   -Wundeclared-selector
 OPTIONS
+
 
 for option in `cat common_options.txt`; do