Skip to content
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

Catch DateTime parsing errors #5

Closed
wants to merge 4 commits into from
Closed

Catch DateTime parsing errors #5

wants to merge 4 commits into from

Conversation

tmaier
Copy link
Contributor

@tmaier tmaier commented Mar 30, 2013

I had the problem that some of my PDF files had a Creation Date according to the american date format.
pdf_info should handle this case.

In the first step, it tries to parse it according to the american format. If this also fails, it just returns nil.

I also used the opportunity to

  1. strip the metadata key from whitespace.
    e.g. foo : bar would return the key foo with the value bar
  2. Wrote some more specs for process_output

@tomtaylor
Copy link
Contributor

Thanks! I rebased and merged this.

@tomtaylor tomtaylor closed this May 16, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants