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

Unicode character classes doxygen #2856

Conversation

erslavin
Copy link
Contributor

Description of Change(s)

Fixes issue with doxygen commenting conventions within unicode character classes, changing from use of @ to \.

Fixes Issue(s)

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

@jesschimein
Copy link
Contributor

Filed as internal issue #USD-9067

@dsyu-pixar
Copy link

@erslavin I'm having a little trouble merging this PR as-is, I think because the first commit adds files that have already been added/merged by other PRs? I think(?) the main change for this PR are the doxygen "@" -> "" changes you made in the second commit of this PR -- can this PR be rebased somehow to just pull in changes from that commit? Otherwise, I can just manually make the same changes.

- Adjusted doxygen style in unicode character classes from @ to \
@erslavin erslavin force-pushed the unicode_character_classes_doxygen branch from b654e10 to e72b756 Compare December 13, 2023 22:14
@pixar-oss pixar-oss merged commit bc8eb4c into PixarAnimationStudios:dev Dec 15, 2023
3 of 5 checks passed
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.

4 participants