On Mon, 09 Feb 2009, [email protected] wrote:

Hi Viktor,

> 2009-02-10 00:39 UTC+0100 Viktor Szakats (harbour.01 syenar hu)
>   * include/hbapigt.h
>   * source/rtl/gtfunc.c
>     + Extended HB_GT_INFO structure with nPCount member.

It's not clear for me why we need this parameter.
So far we were checking for NULL in pNewVal.
Should we now update the whole code which uses pNewVal to 1-st
check if nPCount >= 1 and code which uses pNewVal2 to check
if nPCount >= 2?
I'm finding it redundant and confusing.
I can understand that we may want to introduce such member but
for me it makes sense only if we change the whole HB_GT_INFO
structure to support variable number of parameters instead of
fixed ones. In such case we will probably need some helper
functions or macros to initialize this structure, add new
parameters and clear if necessary.

best regards,
Przemek
_______________________________________________
Harbour mailing list
[email protected]
http://lists.harbour-project.org/mailman/listinfo/harbour

Reply via email to