Re: Observed Compilation warning in WIN32 build

On 2014-01-28 09:13:15 +0000, Rajeev rastogi wrote:
> I observed below WIN32 compilation warnings in postmaster.c (seems introduced by commit
ea9df812d8502fff74e7bc37d61bdc7d66d77a7f"Relax the requirement that all lwlocks be stored in a single array.").
 
> 
>                 1>.\src\backend\postmaster\postmaster.c(5625) : warning C4133: '=' : incompatible types - from
'LWLockPadded*' to 'LWLock *'
 
> 1>.\src\backend\postmaster\postmaster.c(5856) : warning C4133: '=' : incompatible types - from 'LWLock *' to
'LWLockPadded*'
 
> 
> Attached is the patch with the fix.
> 
> Thanks and Regards,
> Kumar Rajeev Rastogi
> 

> *** a/src/backend/postmaster/postmaster.c
> --- b/src/backend/postmaster/postmaster.c
> ***************
> *** 5622,5628 **** save_backend_variables(BackendParameters *param, Port *port,
>   #ifndef HAVE_SPINLOCKS
>       param->SpinlockSemaArray = SpinlockSemaArray;
>   #endif
> !     param->MainLWLockArray = MainLWLockArray;
>       param->ProcStructLock = ProcStructLock;
>       param->ProcGlobal = ProcGlobal;
>       param->AuxiliaryProcs = AuxiliaryProcs;
> --- 5622,5628 ----
>   #ifndef HAVE_SPINLOCKS
>       param->SpinlockSemaArray = SpinlockSemaArray;
>   #endif
> !     param->MainLWLockArray = (LWLock*)MainLWLockArray;
>       param->ProcStructLock = ProcStructLock;
>       param->ProcGlobal = ProcGlobal;
>       param->AuxiliaryProcs = AuxiliaryProcs;
> ***************
> *** 5853,5859 **** restore_backend_variables(BackendParameters *param, Port *port)
>   #ifndef HAVE_SPINLOCKS
>       SpinlockSemaArray = param->SpinlockSemaArray;
>   #endif
> !     MainLWLockArray = param->MainLWLockArray;
>       ProcStructLock = param->ProcStructLock;
>       ProcGlobal = param->ProcGlobal;
>       AuxiliaryProcs = param->AuxiliaryProcs;
> --- 5853,5859 ----
>   #ifndef HAVE_SPINLOCKS
>       SpinlockSemaArray = param->SpinlockSemaArray;
>   #endif
> !     MainLWLockArray = (LWLockPadded*)param->MainLWLockArray;
>       ProcStructLock = param->ProcStructLock;
>       ProcGlobal = param->ProcGlobal;
>       AuxiliaryProcs = param->AuxiliaryProcs;

This strikes me as the wrong fix, the types in BackendParams should be
changed instead.

Greetings,

Andres Freund

-- Andres Freund                       http://www.2ndQuadrant.com/PostgreSQL Development, 24x7 Support, Training &
Services