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

Change in imagetopnm of convert.c in case image->comps[].data is NULL #1156

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Change in imagetopnm of convert.c in case image->comps[].data is NULL #1156

wants to merge 1 commit into from

Conversation

szukw000
Copy link
Contributor

See #1152 for the patch in opj_decompress: FlipDeter does not have an extension.
winfried


fputs("\n===========================================\n", stderr);
#ifdef WIN32
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it is legal to use forward slash on Windows too: c:/users/bla/...

@szukw000
Copy link
Contributor Author

@rouault ,
please see:

opj_decompress.c:
line 449: const char* path_separator = "\";

windirent.h:
static DIR opendir(const char dirname)
line 357: strlen("\
.
")

winfried

@rouault
Copy link
Collaborator

rouault commented Nov 16, 2018

opj_decompress.c:
line 449: const char* path_separator = "";

That's not directly relevant. get_next_file() is free to use backslash to create a filename. But in get_file_format() , you receive a filename from the command line, and the user might have used forward slash.

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.

2 participants