Visualizer: remove glfwTerminate() call from destructor (fix #4960) (See PR #4963) [Author: @bchretien]

[ssheorey]

This is an update of PR #4963 by @bchretien (original PR did not allow maintainer updates and had a merge conflict).

See PR #4963 for details.

Type

• Bug fix (non-breaking change which fixes an issue): Fixes glfwTerminate() called in the Visualizer destructor #4960
• New feature (non-breaking change which adds functionality). Resolves #
• Breaking change (fix or feature that would cause existing functionality to not work as expected) Resolves #

Motivation and Context

Checklist:

• I have run python util/check_style.py --apply to apply Open3D code style
  to my code.
• This PR changes Open3D behavior or adds new functionality.
  • Both C++ (Doxygen) and Python (Sphinx / Google style) documentation is
    updated accordingly.
  • I have added or updated C++ and / or Python unit tests OR included test
    results (e.g. screenshots or numbers) here.
• I will follow up and update the code if CI fails.
• For fork PRs, I have selected Allow edits from maintainers.

Description

[update-docs]

Thanks for submitting this pull request! The maintainers of this repository would appreciate if you could update the CHANGELOG.md based on your changes.

[bchretien]

@ssheorey thanks for updating and merging it! 👍