Current version (5.10.13) cannot create Excel2007 files

Questions and answers on how to use XLSReadWriteII 5.
Post Reply
mdonath
Posts: 23
Joined: Fri Oct 05, 2012 11:36 am

Current version (5.10.13) cannot create Excel2007 files

Post 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
larsa
Site Admin
Posts: 926
Joined: Mon Jun 27, 2005 9:30 pm

Re: Current version (5.10.13) cannot create Excel2007 files

Post 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.
Lars Arvidsson, Axolot Data
mdonath
Posts: 23
Joined: Fri Oct 05, 2012 11:36 am

Re: Current version (5.10.13) cannot create Excel2007 files

Post 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
larsa
Site Admin
Posts: 926
Joined: Mon Jun 27, 2005 9:30 pm

Re: Current version (5.10.13) cannot create Excel2007 files

Post by larsa »

Hello

Can you please point me to the C++ documentation that claims that names in capitals are reserved for macros?
Lars Arvidsson, Axolot Data
mdonath
Posts: 23
Joined: Fri Oct 05, 2012 11:36 am

Re: Current version (5.10.13) cannot create Excel2007 files

Post 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
Post Reply