Skip to content

Fix Coverity reports:#3606

Merged
skef merged 1 commit intofontforge:masterfrom
serval2412:coverity
Mar 30, 2019
Merged

Fix Coverity reports:#3606
skef merged 1 commit intofontforge:masterfrom
serval2412:coverity

Conversation

@serval2412
Copy link
Copy Markdown
Contributor

1399658
1082974
1083593
1082951
1082952
1082953
1083081
1083083
1083086
1083093
1083106
1083115
1083577

Provide a general summary of your changes in the Title above.

Important

Mark with [x] to select. Leave as [ ] to unselect.

Motivation and Context

  • Why is this change required? What problem does it solve?
  • If it fixes an open issue, include the text Closes #1 (where 1 would be the issue number) to your commit message.

Types of changes

What types of changes does your code introduce? Check all the boxes that apply:

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Description

  • Describe your changes in detail.

Final checklist

Go over all the following points and check all the boxes that apply.
If you're unsure about any of these, don't hesitate to ask. We're here to help! Various areas of the codebase have been worked on by different people in recent years, so if you are unfamiliar with the general area you're working in, please feel free to chat with people who have experience in that area. See the list here.

  • My code follows the code style of this project found here.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING guidelines.

Your pull request will be tested via Travis CI to automatically indicate that your changes do not prevent compilation. FontForge is a big program, so Travis can easily take over 20 minutes to confirm your changes are buildable. Please be patient. More details about using Travis can be found here.

If it reports back that there are problems, you can log into the Travis system and check the log report for your pull request to see what the problem was. If no error is shown, just re-run the Travis test for your pull-request (that failed) to see a fresh report since the last report may be for someone else that did a later pull request, or for mainline code. If you add new code to fix your issue/problem, then take note that you need to check the next pull request in the Travis system. Travis issue numbers are different from GitHub issue numbers.

1399658
1082974
1083593
1082951
1082952
1082953
1083081
1083083
1083086
1083093
1083106
1083115
1083577
@skef skef merged commit 701857f into fontforge:master Mar 30, 2019
@serval2412 serval2412 deleted the coverity branch March 30, 2019 12:26
Omnikron13 pushed a commit to Omnikron13/fontforge that referenced this pull request May 31, 2022
1399658
1082974
1083593
1082951
1082952
1082953
1083081
1083083
1083086
1083093
1083106
1083115
1083577
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