USe const compare function pointer arguments
authorArmin Novak <armin.novak@thincast.com>
Thu, 18 Oct 2018 07:06:55 +0000 (09:06 +0200)
committerArmin Novak <armin.novak@thincast.com>
Thu, 18 Oct 2018 07:06:55 +0000 (09:06 +0200)
winpr/include/winpr/collections.h
winpr/libwinpr/thread/thread.c
winpr/libwinpr/utils/collections/ArrayList.c
winpr/libwinpr/utils/collections/LinkedList.c
winpr/libwinpr/utils/collections/ListDictionary.c
winpr/libwinpr/utils/collections/Queue.c
winpr/libwinpr/utils/collections/Stack.c

index af058a7..237e5b3 100644 (file)
@@ -41,7 +41,7 @@ typedef void* (*OBJECT_NEW_FN)();
 typedef void (*OBJECT_INIT_FN)(void* obj);
 typedef void (*OBJECT_UNINIT_FN)(void* obj);
 typedef void (*OBJECT_FREE_FN)(void* obj);
-typedef BOOL (*OBJECT_EQUALS_FN)(void* objA, void* objB);
+typedef BOOL (*OBJECT_EQUALS_FN)(const void* objA, const void* objB);
 
 struct _wObject
 {
@@ -208,16 +208,17 @@ WINPR_API int ListDictionary_Count(wListDictionary* listDictionary);
 WINPR_API void ListDictionary_Lock(wListDictionary* listDictionary);
 WINPR_API void ListDictionary_Unlock(wListDictionary* listDictionary);
 
-WINPR_API BOOL ListDictionary_Add(wListDictionary* listDictionary, void* key, void* value);
-WINPR_API void* ListDictionary_Remove(wListDictionary* listDictionary, void* key);
+WINPR_API BOOL ListDictionary_Add(wListDictionary* listDictionary, const void* key, void* value);
+WINPR_API void* ListDictionary_Remove(wListDictionary* listDictionary, const void* key);
 WINPR_API void* ListDictionary_Remove_Head(wListDictionary* listDictionary);
 WINPR_API void ListDictionary_Clear(wListDictionary* listDictionary);
 
-WINPR_API BOOL ListDictionary_Contains(wListDictionary* listDictionary, void* key);
+WINPR_API BOOL ListDictionary_Contains(wListDictionary* listDictionary, const void* key);
 WINPR_API int ListDictionary_GetKeys(wListDictionary* listDictionary, ULONG_PTR** ppKeys);
 
-WINPR_API void* ListDictionary_GetItemValue(wListDictionary* listDictionary, void* key);
-WINPR_API BOOL ListDictionary_SetItemValue(wListDictionary* listDictionary, void* key, void* value);
+WINPR_API void* ListDictionary_GetItemValue(wListDictionary* listDictionary, const void* key);
+WINPR_API BOOL ListDictionary_SetItemValue(wListDictionary* listDictionary, const void* key,
+        void* value);
 
 WINPR_API wListDictionary* ListDictionary_New(BOOL synchronized);
 WINPR_API void ListDictionary_Free(wListDictionary* listDictionary);
index 0c678ae..fef495f 100644 (file)
@@ -269,10 +269,10 @@ static BOOL reset_event(WINPR_THREAD* thread)
        return status;
 }
 
-static BOOL thread_compare(void* a, void* b)
+static BOOL thread_compare(const void* a, const void* b)
 {
-       pthread_t* p1 = a;
-       pthread_t* p2 = b;
+       const pthread_t* p1 = a;
+       const pthread_t* p2 = b;
        BOOL rc = pthread_equal(*p1, *p2);
        return rc;
 }
index f39cc96..6260550 100644 (file)
@@ -38,7 +38,7 @@
  * Gets or sets the number of elements that the ArrayList can contain.
  */
 
-int ArrayList_Capacity(wArrayList *arrayList)
+int ArrayList_Capacity(wArrayListarrayList)
 {
        return arrayList->capacity;
 }
@@ -47,7 +47,7 @@ int ArrayList_Capacity(wArrayList *arrayList)
  * Gets the number of elements actually contained in the ArrayList.
  */
 
-int ArrayList_Count(wArrayList *arrayList)
+int ArrayList_Count(wArrayListarrayList)
 {
        return arrayList->size;
 }
@@ -66,7 +66,7 @@ int ArrayList_Items(wArrayList* arrayList, ULONG_PTR** ppItems)
  * Gets a value indicating whether the ArrayList has a fixed size.
  */
 
-BOOL ArrayList_IsFixedSized(wArrayList *arrayList)
+BOOL ArrayList_IsFixedSized(wArrayListarrayList)
 {
        return FALSE;
 }
@@ -75,7 +75,7 @@ BOOL ArrayList_IsFixedSized(wArrayList *arrayList)
  * Gets a value indicating whether the ArrayList is read-only.
  */
 
-BOOL ArrayList_IsReadOnly(wArrayList *arrayList)
+BOOL ArrayList_IsReadOnly(wArrayListarrayList)
 {
        return FALSE;
 }
@@ -84,7 +84,7 @@ BOOL ArrayList_IsReadOnly(wArrayList *arrayList)
  * Gets a value indicating whether access to the ArrayList is synchronized (thread safe).
  */
 
-BOOL ArrayList_IsSynchronized(wArrayList *arrayList)
+BOOL ArrayList_IsSynchronized(wArrayListarrayList)
 {
        return arrayList->synchronized;
 }
@@ -93,7 +93,7 @@ BOOL ArrayList_IsSynchronized(wArrayList *arrayList)
  * Lock access to the ArrayList
  */
 
-void ArrayList_Lock(wArrayList *arrayList)
+void ArrayList_Lock(wArrayListarrayList)
 {
        EnterCriticalSection(&arrayList->lock);
 }
@@ -102,7 +102,7 @@ void ArrayList_Lock(wArrayList *arrayList)
  * Unlock access to the ArrayList
  */
 
-void ArrayList_Unlock(wArrayList *arrayList)
+void ArrayList_Unlock(wArrayListarrayList)
 {
        LeaveCriticalSection(&arrayList->lock);
 }
@@ -111,9 +111,9 @@ void ArrayList_Unlock(wArrayList *arrayList)
  * Gets the element at the specified index.
  */
 
-void *ArrayList_GetItem(wArrayList *arrayList, int index)
+void* ArrayList_GetItem(wArrayList* arrayList, int index)
 {
-       void *obj = NULL;
+       voidobj = NULL;
 
        if ((index >= 0) && (index < arrayList->size))
        {
@@ -127,7 +127,7 @@ void *ArrayList_GetItem(wArrayList *arrayList, int index)
  * Sets the element at the specified index.
  */
 
-void ArrayList_SetItem(wArrayList *arrayList, int index, void *obj)
+void ArrayList_SetItem(wArrayList* arrayList, int index, void* obj)
 {
        if ((index >= 0) && (index < arrayList->size))
        {
@@ -143,15 +143,15 @@ void ArrayList_SetItem(wArrayList *arrayList, int index, void *obj)
  * Shift a section of the list.
  */
 
-BOOL ArrayList_Shift(wArrayList *arrayList, int index, int count)
+BOOL ArrayList_Shift(wArrayListarrayList, int index, int count)
 {
        if (count > 0)
        {
                if (arrayList->size + count > arrayList->capacity)
                {
-                       void **newArray;
+                       void** newArray;
                        int newCapacity = arrayList->capacity * arrayList->growthFactor;
-                       newArray = (void **)realloc(arrayList->array, sizeof(void *) * newCapacity);
+                       newArray = (void**)realloc(arrayList->array, sizeof(void*) * newCapacity);
 
                        if (!newArray)
                                return FALSE;
@@ -160,7 +160,8 @@ BOOL ArrayList_Shift(wArrayList *arrayList, int index, int count)
                        arrayList->capacity = newCapacity;
                }
 
-               MoveMemory(&arrayList->array[index + count], &arrayList->array[index], (arrayList->size - index) * sizeof(void *));
+               MoveMemory(&arrayList->array[index + count], &arrayList->array[index],
+                          (arrayList->size - index) * sizeof(void*));
                arrayList->size += count;
        }
        else if (count < 0)
@@ -168,7 +169,7 @@ BOOL ArrayList_Shift(wArrayList *arrayList, int index, int count)
                int chunk = arrayList->size - index + count;
 
                if (chunk > 0)
-                       MoveMemory(&arrayList->array[index], &arrayList->array[index - count], chunk * sizeof(void *));
+                       MoveMemory(&arrayList->array[index], &arrayList->array[index - count], chunk * sizeof(void*));
 
                arrayList->size += count;
        }
@@ -180,7 +181,7 @@ BOOL ArrayList_Shift(wArrayList *arrayList, int index, int count)
  * Removes all elements from the ArrayList.
  */
 
-void ArrayList_Clear(wArrayList *arrayList)
+void ArrayList_Clear(wArrayListarrayList)
 {
        int index;
 
@@ -205,7 +206,7 @@ void ArrayList_Clear(wArrayList *arrayList)
  * Determines whether an element is in the ArrayList.
  */
 
-BOOL ArrayList_Contains(wArrayList *arrayList, void *obj)
+BOOL ArrayList_Contains(wArrayList* arrayList, void* obj)
 {
        int index;
        BOOL rc = FALSE;
@@ -231,7 +232,7 @@ BOOL ArrayList_Contains(wArrayList *arrayList, void *obj)
  * Adds an object to the end of the ArrayList.
  */
 
-int ArrayList_Add(wArrayList *arrayList, void *obj)
+int ArrayList_Add(wArrayList* arrayList, void* obj)
 {
        int index = -1;
 
@@ -240,9 +241,9 @@ int ArrayList_Add(wArrayList *arrayList, void *obj)
 
        if (arrayList->size + 1 > arrayList->capacity)
        {
-               void **newArray;
+               void** newArray;
                int newCapacity = arrayList->capacity * arrayList->growthFactor;
-               newArray = (void **)realloc(arrayList->array, sizeof(void *) * newCapacity);
+               newArray = (void**)realloc(arrayList->array, sizeof(void*) * newCapacity);
 
                if (!newArray)
                        goto out;
@@ -265,7 +266,7 @@ out:
  * Inserts an element into the ArrayList at the specified index.
  */
 
-BOOL ArrayList_Insert(wArrayList *arrayList, int index, void *obj)
+BOOL ArrayList_Insert(wArrayList* arrayList, int index, void* obj)
 {
        BOOL ret = TRUE;
 
@@ -294,7 +295,7 @@ BOOL ArrayList_Insert(wArrayList *arrayList, int index, void *obj)
  * Removes the first occurrence of a specific object from the ArrayList.
  */
 
-BOOL ArrayList_Remove(wArrayList *arrayList, void *obj)
+BOOL ArrayList_Remove(wArrayList* arrayList, void* obj)
 {
        int index;
        BOOL found = FALSE;
@@ -330,7 +331,7 @@ BOOL ArrayList_Remove(wArrayList *arrayList, void *obj)
  * Removes the element at the specified index of the ArrayList.
  */
 
-BOOL ArrayList_RemoveAt(wArrayList *arrayList, int index)
+BOOL ArrayList_RemoveAt(wArrayListarrayList, int index)
 {
        BOOL ret = TRUE;
 
@@ -361,7 +362,7 @@ BOOL ArrayList_RemoveAt(wArrayList *arrayList, int index)
  * in the ArrayList that contains the specified number of elements and ends at the specified index.
  */
 
-int ArrayList_IndexOf(wArrayList *arrayList, void *obj, int startIndex, int count)
+int ArrayList_IndexOf(wArrayList* arrayList, void* obj, int startIndex, int count)
 {
        int index;
        BOOL found = FALSE;
@@ -403,7 +404,7 @@ int ArrayList_IndexOf(wArrayList *arrayList, void *obj, int startIndex, int coun
  * in the ArrayList that contains the specified number of elements and ends at the specified index.
  */
 
-int ArrayList_LastIndexOf(wArrayList *arrayList, void *obj, int startIndex, int count)
+int ArrayList_LastIndexOf(wArrayList* arrayList, void* obj, int startIndex, int count)
 {
        int index;
        BOOL found = FALSE;
@@ -435,7 +436,7 @@ int ArrayList_LastIndexOf(wArrayList *arrayList, void *obj, int startIndex, int
        return index;
 }
 
-static BOOL ArrayList_DefaultCompare(void *objA, void *objB)
+static BOOL ArrayList_DefaultCompare(const void* objA, const void* objB)
 {
        return objA == objB ? TRUE : FALSE;
 }
@@ -444,10 +445,10 @@ static BOOL ArrayList_DefaultCompare(void *objA, void *objB)
  * Construction, Destruction
  */
 
-wArrayList *ArrayList_New(BOOL synchronized)
+wArrayListArrayList_New(BOOL synchronized)
 {
-       wArrayList *arrayList = NULL;
-       arrayList = (wArrayList *)calloc(1, sizeof(wArrayList));
+       wArrayListarrayList = NULL;
+       arrayList = (wArrayList*)calloc(1, sizeof(wArrayList));
 
        if (!arrayList)
                return NULL;
@@ -456,7 +457,7 @@ wArrayList *ArrayList_New(BOOL synchronized)
        arrayList->capacity = 32;
        arrayList->growthFactor = 2;
        arrayList->object.fnObjectEquals = ArrayList_DefaultCompare;
-       arrayList->array = (void **)calloc(arrayList->capacity, sizeof(void *));
+       arrayList->array = (void**)calloc(arrayList->capacity, sizeof(void*));
 
        if (!arrayList->array)
                goto out_free;
@@ -468,7 +469,7 @@ out_free:
        return NULL;
 }
 
-void ArrayList_Free(wArrayList *arrayList)
+void ArrayList_Free(wArrayListarrayList)
 {
        if (!arrayList)
                return;
index fee1e80..03dbe7c 100644 (file)
@@ -320,7 +320,7 @@ BOOL LinkedList_Enumerator_MoveNext(wLinkedList* list)
        return TRUE;
 }
 
-static BOOL default_equal_function(void* objA, void* objB)
+static BOOL default_equal_function(const void* objA, const void* objB)
 {
        return objA == objB;
 }
index 40cb72b..818d513 100644 (file)
@@ -155,7 +155,8 @@ int ListDictionary_GetKeys(wListDictionary* listDictionary, ULONG_PTR** ppKeys)
 
        if (count)
        {
-               pKeys = (ULONG_PTR *) calloc(count, sizeof(ULONG_PTR));
+               pKeys = (ULONG_PTR*) calloc(count, sizeof(ULONG_PTR));
+
                if (!pKeys)
                {
                        if (listDictionary->synchronized)
@@ -190,7 +191,7 @@ int ListDictionary_GetKeys(wListDictionary* listDictionary, ULONG_PTR** ppKeys)
  * Adds an entry with the specified key and value into the ListDictionary.
  */
 
-BOOL ListDictionary_Add(wListDictionary* listDictionary, void* key, void* value)
+BOOL ListDictionary_Add(wListDictionary* listDictionary, const void* key, void* value)
 {
        wListDictionaryItem* item;
        wListDictionaryItem* lastItem;
@@ -203,12 +204,12 @@ BOOL ListDictionary_Add(wListDictionary* listDictionary, void* key, void* value)
                EnterCriticalSection(&listDictionary->lock);
 
        item = (wListDictionaryItem*) malloc(sizeof(wListDictionaryItem));
+
        if (!item)
                goto out_error;
 
-       item->key = key;
+       item->key = (void*)key;
        item->value = value;
-
        item->next = NULL;
 
        if (!listDictionary->head)
@@ -227,6 +228,7 @@ BOOL ListDictionary_Add(wListDictionary* listDictionary, void* key, void* value)
 
        ret = TRUE;
 out_error:
+
        if (listDictionary->synchronized)
                LeaveCriticalSection(&listDictionary->lock);
 
@@ -277,7 +279,7 @@ void ListDictionary_Clear(wListDictionary* listDictionary)
  * Determines whether the ListDictionary contains a specific key.
  */
 
-BOOL ListDictionary_Contains(wListDictionary* listDictionary, void* key)
+BOOL ListDictionary_Contains(wListDictionary* listDictionary, const void* key)
 {
        wListDictionaryItem* item;
        OBJECT_EQUALS_FN keyEquals;
@@ -309,7 +311,7 @@ BOOL ListDictionary_Contains(wListDictionary* listDictionary, void* key)
  * Removes the entry with the specified key from the ListDictionary.
  */
 
-void* ListDictionary_Remove(wListDictionary* listDictionary, void* key)
+void* ListDictionary_Remove(wListDictionary* listDictionary, const void* key)
 {
        void* value = NULL;
        wListDictionaryItem* item;
@@ -323,7 +325,6 @@ void* ListDictionary_Remove(wListDictionary* listDictionary, void* key)
                EnterCriticalSection(&listDictionary->lock);
 
        keyEquals = listDictionary->objectKey.fnObjectEquals;
-
        item = listDictionary->head;
        prevItem = NULL;
 
@@ -335,6 +336,7 @@ void* ListDictionary_Remove(wListDictionary* listDictionary, void* key)
                                listDictionary->head = item->next;
                        else
                                prevItem->next = item->next;
+
                        value = item->value;
                        free(item);
                        break;
@@ -354,10 +356,10 @@ void* ListDictionary_Remove(wListDictionary* listDictionary, void* key)
  * Removes the first (head) entry from the list
  */
 
-void *ListDictionary_Remove_Head(wListDictionary* listDictionary)
+voidListDictionary_Remove_Head(wListDictionary* listDictionary)
 {
        wListDictionaryItem* item;
-       void *value = NULL;
+       voidvalue = NULL;
 
        if (!listDictionary)
                return NULL;
@@ -375,6 +377,7 @@ void *ListDictionary_Remove_Head(wListDictionary* listDictionary)
 
        if (listDictionary->synchronized)
                LeaveCriticalSection(&listDictionary->lock);
+
        return value;
 }
 
@@ -382,7 +385,7 @@ void *ListDictionary_Remove_Head(wListDictionary* listDictionary)
  * Get an item value using key
  */
 
-void* ListDictionary_GetItemValue(wListDictionary* listDictionary, void* key)
+void* ListDictionary_GetItemValue(wListDictionary* listDictionary, const void* key)
 {
        void* value = NULL;
        wListDictionaryItem* item = NULL;
@@ -421,7 +424,7 @@ void* ListDictionary_GetItemValue(wListDictionary* listDictionary, void* key)
  * Set an item value using key
  */
 
-BOOL ListDictionary_SetItemValue(wListDictionary* listDictionary, void* key, void* value)
+BOOL ListDictionary_SetItemValue(wListDictionary* listDictionary, const void* key, void* value)
 {
        BOOL status = FALSE;
        wListDictionaryItem* item;
@@ -434,6 +437,7 @@ BOOL ListDictionary_SetItemValue(wListDictionary* listDictionary, void* key, voi
                EnterCriticalSection(&listDictionary->lock);
 
        keyEquals = listDictionary->objectKey.fnObjectEquals;
+
        if (listDictionary->head)
        {
                item = listDictionary->head;
@@ -450,6 +454,7 @@ BOOL ListDictionary_SetItemValue(wListDictionary* listDictionary, void* key, voi
                {
                        if (listDictionary->objectValue.fnObjectFree)
                                listDictionary->objectValue.fnObjectFree(item->value);
+
                        item->value = value;
                }
 
@@ -462,7 +467,7 @@ BOOL ListDictionary_SetItemValue(wListDictionary* listDictionary, void* key, voi
        return status;
 }
 
-static BOOL default_equal_function(void *obj1, void *obj2)
+static BOOL default_equal_function(const void* obj1, const void* obj2)
 {
        return (obj1 == obj2);
 }
@@ -473,7 +478,6 @@ static BOOL default_equal_function(void *obj1, void *obj2)
 wListDictionary* ListDictionary_New(BOOL synchronized)
 {
        wListDictionary* listDictionary = NULL;
-
        listDictionary = (wListDictionary*) calloc(1, sizeof(wListDictionary));
 
        if (!listDictionary)
@@ -489,7 +493,6 @@ wListDictionary* ListDictionary_New(BOOL synchronized)
 
        listDictionary->objectKey.fnObjectEquals = default_equal_function;
        listDictionary->objectValue.fnObjectEquals = default_equal_function;
-
        return listDictionary;
 }
 
index d6f8d1b..2e8abd6 100644 (file)
@@ -41,6 +41,7 @@
 int Queue_Count(wQueue* queue)
 {
        int ret;
+
        if (queue->synchronized)
                EnterCriticalSection(&queue->lock);
 
@@ -48,6 +49,7 @@ int Queue_Count(wQueue* queue)
 
        if (queue->synchronized)
                LeaveCriticalSection(&queue->lock);
+
        return ret;
 }
 
@@ -150,12 +152,11 @@ BOOL Queue_Enqueue(wQueue* queue, void* obj)
        {
                int old_capacity;
                int new_capacity;
-               void **newArray;
-
+               void** newArray;
                old_capacity = queue->capacity;
                new_capacity = queue->capacity * queue->growthFactor;
+               newArray = (void**)realloc(queue->array, sizeof(void*) * new_capacity);
 
-               newArray = (void **)realloc(queue->array, sizeof(void*) * new_capacity);
                if (!newArray)
                {
                        ret = FALSE;
@@ -177,12 +178,12 @@ BOOL Queue_Enqueue(wQueue* queue, void* obj)
        queue->array[queue->tail] = obj;
        queue->tail = (queue->tail + 1) % queue->capacity;
        queue->size++;
-
        SetEvent(queue->event);
-
 out:
+
        if (queue->synchronized)
                LeaveCriticalSection(&queue->lock);
+
        return ret;
 }
 
@@ -234,7 +235,7 @@ void* Queue_Peek(wQueue* queue)
        return obj;
 }
 
-static BOOL default_queue_equals(void *obj1, void *obj2)
+static BOOL default_queue_equals(const void* obj1, const void* obj2)
 {
        return (obj1 == obj2);
 }
@@ -246,14 +247,13 @@ static BOOL default_queue_equals(void *obj1, void *obj2)
 wQueue* Queue_New(BOOL synchronized, int capacity, int growthFactor)
 {
        wQueue* queue = NULL;
+       queue = (wQueue*)calloc(1, sizeof(wQueue));
 
-       queue = (wQueue *)calloc(1, sizeof(wQueue));
        if (!queue)
                return NULL;
 
        queue->capacity = 32;
        queue->growthFactor = 2;
-
        queue->synchronized = synchronized;
 
        if (capacity > 0)
@@ -262,11 +262,13 @@ wQueue* Queue_New(BOOL synchronized, int capacity, int growthFactor)
        if (growthFactor > 0)
                queue->growthFactor = growthFactor;
 
-       queue->array = (void **)calloc(queue->capacity, sizeof(void *));
+       queue->array = (void**)calloc(queue->capacity, sizeof(void*));
+
        if (!queue->array)
                goto out_free;
 
        queue->event = CreateEvent(NULL, TRUE, FALSE, NULL);
+
        if (!queue->event)
                goto out_free_array;
 
@@ -275,7 +277,6 @@ wQueue* Queue_New(BOOL synchronized, int capacity, int growthFactor)
 
        queue->object.fnObjectEquals = default_queue_equals;
        return queue;
-
 out_free_event:
        CloseHandle(queue->event);
 out_free_array:
@@ -291,7 +292,6 @@ void Queue_Free(wQueue* queue)
                return;
 
        Queue_Clear(queue);
-
        CloseHandle(queue->event);
        DeleteCriticalSection(&queue->lock);
        free(queue->array);
index 14b2d79..a47ab17 100644 (file)
@@ -128,12 +128,13 @@ void Stack_Push(wStack* stack, void* obj)
        if ((stack->size + 1) >= stack->capacity)
        {
                int new_cap;
-               void **new_arr;
-
+               void** new_arr;
                new_cap = stack->capacity * 2;
                new_arr = (void**) realloc(stack->array, sizeof(void*) * new_cap);
+
                if (!new_arr)
                        return;
+
                stack->array = new_arr;
                stack->capacity = new_cap;
        }
@@ -185,7 +186,7 @@ void* Stack_Peek(wStack* stack)
 }
 
 
-static BOOL default_stack_equals(void *obj1, void *obj2)
+static BOOL default_stack_equals(const void* obj1, const void* obj2)
 {
        return (obj1 == obj2);
 }
@@ -197,24 +198,23 @@ static BOOL default_stack_equals(void *obj1, void *obj2)
 wStack* Stack_New(BOOL synchronized)
 {
        wStack* stack = NULL;
+       stack = (wStack*)calloc(1, sizeof(wStack));
 
-       stack = (wStack *)calloc(1, sizeof(wStack));
        if (!stack)
                return NULL;
 
        stack->object.fnObjectEquals = default_stack_equals;
        stack->synchronized = synchronized;
-
        stack->capacity = 32;
        stack->array = (void**) calloc(stack->capacity, sizeof(void*));
+
        if (!stack->array)
                goto out_free;
 
        if (stack->synchronized && !InitializeCriticalSectionAndSpinCount(&stack->lock, 4000))
-                       goto out_free_array;
+               goto out_free_array;
 
        return stack;
-
 out_free_array:
        free(stack->array);
 out_free:
@@ -230,7 +230,6 @@ void Stack_Free(wStack* stack)
                        DeleteCriticalSection(&stack->lock);
 
                free(stack->array);
-
                free(stack);
        }
 }