Page 1 of 1

Current version (5.10.13) cannot create Excel2007 files

Posted: Tue Apr 09, 2013 7:28 am
by mdonath
The current version is not able to create Excel2007 files. This error did exist a couple of versions before but then it vanished. Not it is here again:

Code: Select all

TDateTime dt(TDateTime::CurrentDate());
for(int i=0; i<Lines; i++)
    XLSReadWriteII51->Sheets[0]->AsDateTime[0][i] = dt + double(i+1)/96.;
XLSReadWriteII51->Filename = L"test01.xlsx";
XLSReadWriteII51->Write();
By the way:

TXLSTokenizer.EOF still exists!

Markus

Re: Current version (5.10.13) cannot create Excel2007 files

Posted: Tue Apr 09, 2013 3:26 pm
by larsa
Hello

There was some debug code in 5.10.13 that caused this problem. Fixed in 5.10.14

TXLSTokenizer.EOF is a valid C++ name and it's not a problem.

Re: Current version (5.10.13) cannot create Excel2007 files

Posted: Tue Apr 09, 2013 3:46 pm
by mdonath
No, that's definitively not correct. Names that contain capitals only are reserved for macros in C++. And a macro EOF exists!

On March 13 you wrote:
"The EOF method is removed. Was not used anyway."

Markus

Re: Current version (5.10.13) cannot create Excel2007 files

Posted: Tue Apr 09, 2013 4:18 pm
by larsa
Hello

Can you please point me to the C++ documentation that claims that names in capitals are reserved for macros?

Re: Current version (5.10.13) cannot create Excel2007 files

Posted: Wed Apr 10, 2013 8:51 am
by mdonath
There is not a strict prohibition for the use of qualifiers in capitals. But it is common sense.
Try to name a qualifier ASSERT or ERROR, you will get funny error messages.

On the other hand I have no problems to comment the EOF line out in every new version of XSLSReadWrite.

Markus