× The internal search function is temporarily non-functional. The current search engine is no longer viable and we are researching alternatives.
As a stop gap measure, we are using Google's custom search engine service.
If you know of an easy to use, open source, search engine ... please contact support@midrange.com.



Hi Jevgeni

Thank you for the correction. I have modified my code.
I should have known this but just shows I need more practice.
I believe I have learnt now there is nothing magic about C functions.
If the string variable is not null terminated you need to handle lengths
explicitly.
Seems that strcmp and sizeof need null terminated strings to work.
Also the original code mostly worked Ok, because the row and col are 1 byte
values and I
did not have more than one field in the same place.

Regards
Frank

date: Tue, 23 Sep 2014 12:13:54 +0300
from: Jevgeni Astanovski <jevgeniast@****.com>
This part of code is invalid, I think:
-snip
If understand correctly your intention, you wanted to compare 2 record
names, 2 field names, row and column of the 2 fields.
But in fact I think that you only compare the first character of the 2
records and the first character of the field names as
sizeof(*p_iRecord) equals to 1.
I would write it so:
if (!memcmp(Functn , "#NEXT" , 5) &&
!memcmp(p_iRecord, Odata->RecordName, 10) &&
!memcmp(p_iField, Odata->FldNam, 10 ) &&
(*p_iRow == *Odata->ROW) &&
(*p_iCol == *Odata->COL))
*GotIt = 1;

As an Amazon Associate we earn from qualifying purchases.

This thread ...


Follow On AppleNews
Return to Archive home page | Return to MIDRANGE.COM home page

This mailing list archive is Copyright 1997-2024 by midrange.com and David Gibbs as a compilation work. Use of the archive is restricted to research of a business or technical nature. Any other uses are prohibited. Full details are available on our policy page. If you have questions about this, please contact [javascript protected email address].

Operating expenses for this site are earned using the Amazon Associate program and Google Adsense.