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

APPLE: Fixes for small leaks #2085

Closed

Conversation

creijon
Copy link
Contributor

@creijon creijon commented Nov 9, 2022

Description of Change(s)

Need to explicitly release the compiler options and descriptors.
Found leaks in the MTLSamplerDescriptor since they needed release too.

Fixes Issue(s)

  • Small memory leaks
  • I have verified that all unit tests pass with the proposed changes
  • I have submitted a signed Contributor License Agreement

@sunyab
Copy link
Contributor

sunyab commented Nov 19, 2022

Filed as internal issue #USD-7770

@creijon creijon force-pushed the jon/dev/small_leaks branch 4 times, most recently from 09a0bc8 to e8c5b4a Compare November 29, 2022 11:41
@creijon creijon force-pushed the jon/dev/small_leaks branch 2 times, most recently from 55995cf to 7f2a460 Compare December 3, 2022 06:59
pixar-oss added a commit that referenced this pull request Dec 9, 2022
APPLE: Fixes for small leaks

(Internal change: 2256955)
@creijon creijon closed this Dec 10, 2022
@creijon creijon deleted the jon/dev/small_leaks branch December 10, 2022 07:44
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