Skip to content
Snippets Groups Projects

vkd3d-shader: Fix a couple of memory leaks.

Merged Giovanni Mascellani requested to merge giomasce/vkd3d:yellowstone into master
2 unresolved threads

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
717 717 int ret;
718 718
719 719 if (!parse_command_line(argc, argv, &options))
720 return 1;
720 goto done;
721 721
722 722 if (options.print_help)
723 723 {
724 724 print_usage(argv[0]);
725 return 0;
725 fail = 0;
  • Giovanni Mascellani approved this merge request

    approved this merge request

  • Francisco Casas approved this merge request

    approved this merge request

  • Henri Verbeet approved this merge request

    approved this merge request

  • One (perhaps obvious) thing to point out here is that main() is starting to become large enough that moving some parts into separate functions would likely simplify error handling a bit.

  • Alexandre Julliard approved this merge request

    approved this merge request

  • added 6 commits

    • 494507bb...d38221bd - 3 commits from branch wine:master
    • 896fc0c2 - vkd3d-compiler: Exit the main function through a common code path.
    • 8b11b4b3 - vkd3d-compiler: Free compilation options (Valgrind).
    • cc72a8d3 - vkd3d-shader/spirv: Free binary SPIR-V code (Valgrind).

    Compare with previous version

  • Please register or sign in to reply
    Loading