RE: [Harbour] to przmek about ordkeyval length

2008-09-19 Thread Massimo Belgrano
Developer List. Subject: Re: [Harbour] to przmek about ordkeyval length > Hi Miguel, > (remember that OrdKeyVal and others doesn't exists at CL52 as you > know perfectly) You've added two new RTEs in CDX code in two commits. Both of these went into the index creation code (hb_

Re: [Harbour] to przmek about ordkeyval length

2008-09-19 Thread Szakáts Viktor
Hi Miguel, (remember that OrdKeyVal and others doesn't exists at CL52 as you know perfectly) You've added two new RTEs in CDX code in two commits. Both of these went into the index creation code (hb_cdxOrderCreate()), so I'm not sure what do these have to do with 5.3-only OrdKeyVal(). What

Re: [Harbour] to przmek about ordkeyval length

2008-09-19 Thread Miguel Angel Marchuet
Szakáts Viktor escribió: Hi Miguel, we are talking about CDX not six. the question is create incompatible CDX or not create ? Incompatible with what? I don't really understand what existing bugs are you trying to fix here to be honest. (remember that OrdKeyVal and others doesn't exists a

Re: [Harbour] to przmek about ordkeyval length

2008-09-19 Thread Szakáts Viktor
Hi Miguel, we are talking about CDX not six. the question is create incompatible CDX or not create ? Incompatible with what? I don't really understand what existing bugs are you trying to fix here to be honest. Instead of revert, i think is better to work by setting. Then people as you tha

Re: [Harbour] to przmek about ordkeyval length

2008-09-19 Thread Miguel Angel Marchuet
we are talking about CDX not six. the question is create incompatible CDX or not create ? Instead of revert, i think is better to work by setting. Then people as you that prefer cl52 compatible or people that prefer cl53 compatible if you want cl52 compatible then you need to break Round functi

Re: Re: [Harbour] to przmek about ordkeyval length

2008-09-19 Thread [EMAIL PROTECTED]
>As Przemek says, SIX for example doesn't RTE. So this seem >rather an RDD driver (and/or version) specific property, >and as such I'd suggest to replicate it that way in Harbour. FYI ADS doesn't RTE too. Regards, Toninho. __ Faça ligações para ou

Re: [Harbour] to przmek about ordkeyval length

2008-09-19 Thread Szakáts Viktor
Hi Miguel, Only CL53/COMIX CDX driver generate RT error for keys longer then maximal supported length. All other like NTX, CL52CDX, SIX3 (NTX,CDX,NSX) and [x]Harbour current code simply cut the index keys to the maximal length without any RT errors so I do not know it's good idea. People

Re: [Harbour] to przmek about ordkeyval length

2008-09-19 Thread Miguel Angel Marchuet
Only CL53/COMIX CDX driver generate RT error for keys longer then maximal supported length. All other like NTX, CL52CDX, SIX3 (NTX,CDX,NSX) and [x]Harbour current code simply cut the index keys to the maximal length without any RT errors so I do not know it's good idea. People may have code which

Re: [Harbour] to przmek about ordkeyval length

2008-09-19 Thread Miguel Angel Marchuet
Only CL53/COMIX CDX driver generate RT error for keys longer then maximal supported length. All other like NTX, CL52CDX, SIX3 (NTX,CDX,NSX) and [x]Harbour current code simply cut the index keys to the maximal length without any RT errors so I do not know it's good idea. People may have code which

Re: [Harbour] to przmek about ordkeyval length

2008-09-19 Thread Szakáts Viktor
Hi Miguel, Przemek, ? "Length of OrdKeyVal() :" + Str( Len(OrdKeyVal()) ) WHILE ! EOF() ? Right( FIELD->Test, 1 ) DbSkip() ENDDO USE FERASE( "__MYTEST.DBF" ) FERASE( "__MYTEST.CDX" ) ? "end" RETURN NIL Only CL53/COMIX CDX driver generate RT error for keys longe

Re: [Harbour] to przmek about ordkeyval length

2008-09-19 Thread Przemyslaw Czerpak
On Fri, 19 Sep 2008, Miguel Angel Marchuet wrote: > OrdKeyVal can't be more large than 240 at cdx almoust in clipper. Just like in all other languages. 240 bytes is forced by internal CDX structure for 512 bytes index page length. You have to be able to store in the worst case at least two keys in

Re: [Harbour] to przmek about ordkeyval length

2008-09-19 Thread Miguel Angel Marchuet
or better /* Make sure KEY has proper type and iLen lower than 240 */ if ( uiLen > CDX_MAXKEY ) { if( hb_cdxErrorRT( pArea, bType == 'U' ? EG_DATATYPE : EG_DATAWIDTH, EDBF_INVALIDKEY, NULL, 0, uiLen > CDX_MAXKEY ? EF_CANDEFAULT : 0 ) == FAILURE ) {