tfussell / xlnt

:bar_chart: Cross-platform user-friendly xlsx library for C++11+
Other
1.5k stars 423 forks source link

date.hpp missing from xlnt.hpp #38

Closed Malvineous closed 8 years ago

Malvineous commented 8 years ago

Hi again,

Been a while but getting back to this now. Just tried compiling with the latest version and getting this error:

import.cpp:450:69: error: invalid use of incomplete type ‘struct xlnt::date’
    auto dt = row[i].get_value<xlnt::date>();
                                                                     ^
In file included from /usr/include/xlnt/xlnt.hpp:26:0,
                 from import.hpp:9,
                 from import.cpp:7:
/usr/include/xlnt/cell/cell.hpp:49:8: note: forward declaration of ‘struct xlnt::date’
 struct date;
        ^

This is fixed by adding an #include for utils/date.hpp into xlnt.hpp. Hopefully you can add this!

tfussell commented 8 years ago

Hi Malvineous,

Thanks for identifying this problem. It always surprises me which platforms have trouble finding headers. I do need to tidy up that main header here at some point.

Malvineous commented 8 years ago

Great, many thanks!