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

Function return value problem #9515

Closed
liyanan1001 opened this issue Aug 16, 2022 · 3 comments
Closed

Function return value problem #9515

liyanan1001 opened this issue Aug 16, 2022 · 3 comments

Comments

@liyanan1001
Copy link

liyanan1001 commented Aug 16, 2022

Describe the bug
src/doxygen.cpp: in the readConfiguration fuction:

case 'w':
        formatName=getArg(argc,argv,optInd);
        if (formatName.isEmpty())
        {
          err("option \"-w\" is missing format specifier rtf, html or latex\n");
          cleanUpDoxygen();
          exit(1);
        }
        if (qstricmp(formatName.data(),"rtf")==0)
        {
          if (optInd+1>=argc)
          {
            err("option \"-w rtf\" is missing a style sheet file name\n");
            cleanUpDoxygen();
            exit(1);
          }
          std::ofstream f;
          if (openOutputFile(argv[optInd+1],f))
          {
            TextStream t(&f);
            RTFGenerator::writeStyleSheetFile(t);
          }
          cleanUpDoxygen();
          exit(1);
        }

doxygen -w rtf ,the command is executed successfully, the value 1 is returned

Expected behavior
doxygen -w rtf , if the execution is successful, the return value is 0

albert-github added a commit to albert-github/doxygen that referenced this issue Aug 16, 2022
Give waring in case file cannot be opened and exit with error return value otherwise exit without error return value
@albert-github
Copy link
Collaborator

I've just pushed a proposed patch, pull request #9516

doxygen added a commit that referenced this issue Aug 17, 2022
@albert-github albert-github added the fixed but not released Bug is fixed in github, but still needs to make its way to an official release label Aug 17, 2022
@albert-github
Copy link
Collaborator

Code has been integrated in master on GitHub (please don't close the issue as this will be done at the moment of an official release).

@doxygen
Copy link
Owner

doxygen commented Aug 26, 2022

This issue was previously marked 'fixed but not released',
which means it should be fixed in doxygen version 1.9.5.
Please verify if this is indeed the case. Reopen the
issue if you think it is not fixed and please include any additional information
that you think can be relevant (preferably in the form of a self-contained example).

@doxygen doxygen removed the fixed but not released Bug is fixed in github, but still needs to make its way to an official release label Aug 26, 2022
@doxygen doxygen closed this as completed Aug 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants