-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
'Compression' feature #10
Conversation
const string GPS_DESC = "gps"; | ||
const string GPS_COORD_LAT_FIELD = "lat"; | ||
const string GPS_COORD_LNG_FIELD = "lng"; | ||
const string GPS_TIME_FIELD = "Time"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is an optional timestamp attribute within every metadata item, let's use it instead.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
@apavlenko , all your comments until this moment are fixed, please check the solutions |
{ | ||
string algo; | ||
compressedXMP->GetProperty(VMF_NS, compressionAlgoPropName.c_str(), &algo, NULL); | ||
if(!algo.empty()) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
and what else
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed, throw an exception if no algo name is given
22659d7
to
fbd8e7c
Compare
void XMPDataSource::loadXMPstructs() | ||
{ | ||
std::shared_ptr<SXMPMeta> compressedXMP = make_shared<SXMPMeta>(); | ||
//xmpFile.GetXMP(NULL, &buffer, NULL); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let's cleanup replaced code, VCS should keep it, not comments
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
…ter classed added, WIP
…stead of parseAll()
30756b5
to
82d410a
Compare
👍 |
1 similar comment
👍 |
See #2 for details