Cleanup duplicate code: add function FreeGroupWithArgs

Signed-off-by: Samuel Ghinet <sghinet@cloudbasesolutions.com>
Cleanup duplicate code: add function _FlowTable_Free

Signed-off-by: Samuel Ghinet <sghinet@cloudbasesolutions.com>
Remove func: DestroyArgumentsFromGroup

Its functionality is virtually identical to DestroyArguments.
Cleanup: "Cleanup" sections of FlowToMessage.c

Signed-off-by: Samuel Ghinet <sghinet@cloudbasesolutions.com>
Refactor: merge DestroyArgList and FreeArgList

The functionality they have is virtually identical one to the other.

Signed-off-by: Samuel Ghinet <sghinet@cloudbasesolutions.com>
Cleanup OvsRefCount.h

Instead of:
if (cond)
{
    // a very lot of code
}

do:
if (!cond)
{
    return;
}

// a very lot of code
Cleanup usages of OVS_REFCOUNT_DEREFERENCE_ONLY

Signed-off-by: Samuel Ghinet <sghinet@cloudbasesolutions.com>
cleanup usages of OVS_REFCOUNT_DEREFERENCE

OVS_REFCOUNT_DEREFERENCE(p) does both:
o) check p: if it's null, it does nothing
o) at the end sets p to NULL

So when using OVS_REFCOUNT_DEREFERENCE, additional checks,
or later sets of p to NULL are not needed.

Signed-off-by: Samuel Ghinet <sghinet@cloudbasesolutions.com>
Cleanup usage of OVS_REFCOUNT_DESTROY

OVS_REFCOUNT_DESTROY(p) checks if p != NULL, so there's no need
of checking p before calling OVS_REFCOUNT_DESTROY.
Cleanup Driver.c

NOTE:
o) Driver_Uninit is not currently used.
o) Driver_Uninit used OVS_REFCOUNT_DEREF_AND_DESTROY(pDatapath)
when it should have used OVS_REFCOUNT_DESTROY(pDatapath)
Cleanup: NdisFilter.c

Also rename variable names, to preserve coding style.

Signed-off-by: Samuel Ghinet <sghinet@cloudbasesolutions.com>
Cleanup OvsCore.h

IfCountedStringToWCharArray is no longer used.
Cleanup duplicates: add func DestroyMessages

Functionality that destroys an array of OVS_MESSAGE structs is virtually
identical in two places in code. We need to extract a function of the two.

Signed-off-by: Samuel Ghinet <sghinet@cloudbasesolutions.com>
Fix: static function declared in header

_ParseArgGroup_FromAttributes is declared in Message.h. It is used in
Message.c only. We need to move it in Message.c

Signed-off-by: Samuel Ghinet <sghinet@cloudbasesolutions.com>