> > [...]
> > -       LWLockInitialize(&dsm->lck, 
> > LWLockNewTrancheId("test_dsm_registry"));
> > +       LWLockInitialize(&dsm->lck, LWLockNewTrancheId((char *) arg));
> >         dsm->val = 0;
> > [...]
> >
> > instead of creating a new test? For the other GetNamedDSMSegment calls,
> > I'll pass NULL to the void * and hard code the tranche name in the init
> > callback.
>
> I think we should verify the pointer value more directly.  For example, we
> could pass something like (uintptr_t) 0x12345 via the callback argument and
> then verify it's the same in the callback.

This is better than what I had earlier where I was testing that a string
was passed correctly. See v6.

--
Sami Imseih
Amazon Web Services(AWS)

Attachment: v6-0001-Add-init_callback_arg-parameter-to-GetNamedDSMSeg.patch
Description: Binary data

Reply via email to