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

fix anon-struct usage that's a warning/error -Wnon-c-typedef-for-linkage #137

Closed
wants to merge 1 commit into from

Conversation

luciang
Copy link
Contributor

@luciang luciang commented Apr 30, 2022

Summary:
Fix
error: anonymous non-C-compatible type given name for linkage purposes by alias declaration; add a tag name here [-Werror,-Wnon-c-typedef-for-linkage]

Reviewed By: philippv

Differential Revision: D36043476

@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported labels Apr 30, 2022
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D36043476

luciang added a commit to luciang/fbzmq that referenced this pull request Apr 30, 2022
…age (#137)

Summary:
X-link: facebook/CacheLib#137

Fix
  error: anonymous non-C-compatible type given name for linkage purposes by alias declaration; add a tag name here [-Werror,-Wnon-c-typedef-for-linkage]

Reviewed By: philippv

Differential Revision: D36043476

fbshipit-source-id: c4b0203083c496c1fac4106fb500e553f3d4b5b4
luciang added a commit to luciang/gloo that referenced this pull request Apr 30, 2022
…age (facebookincubator#137)

Summary:
X-link: facebook/CacheLib#137

Fix
  error: anonymous non-C-compatible type given name for linkage purposes by alias declaration; add a tag name here [-Werror,-Wnon-c-typedef-for-linkage]

Reviewed By: philippv

Differential Revision: D36043476

fbshipit-source-id: c315a11e863b413a7662a497a3db71147488e782
luciang added a commit to luciang/CacheLib that referenced this pull request Apr 30, 2022
…age (facebook#137)

Summary:
Pull Request resolved: facebook#137

Fix
  error: anonymous non-C-compatible type given name for linkage purposes by alias declaration; add a tag name here [-Werror,-Wnon-c-typedef-for-linkage]

Reviewed By: philippv

Differential Revision: D36043476

fbshipit-source-id: 45e2110b007e102388d1487a7f071de440d0bed3
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D36043476

…age (#76610)

Summary:
X-link: pytorch/pytorch#76610

X-link: facebookincubator/gloo#328

X-link: facebookarchive/fbzmq#38

Pull Request resolved: facebook#137

Fix
  error: anonymous non-C-compatible type given name for linkage purposes by alias declaration; add a tag name here [-Werror,-Wnon-c-typedef-for-linkage]

Reviewed By: philippv

Differential Revision: D36043476

fbshipit-source-id: d26f1bf9952b4371d3d4901ae15a68af8fd4c2a1
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D36043476

luciang added a commit to luciang/gloo that referenced this pull request Apr 30, 2022
…age (#76610)

Summary:
X-link: pytorch/pytorch#76610

Pull Request resolved: facebookincubator#328

X-link: facebookarchive/fbzmq#38

X-link: facebook/CacheLib#137

Fix
  error: anonymous non-C-compatible type given name for linkage purposes by alias declaration; add a tag name here [-Werror,-Wnon-c-typedef-for-linkage]

Reviewed By: philippv

Differential Revision: D36043476

fbshipit-source-id: 6e1dc16bf1d205bfcf1a359f0e812a9bfdb0d039
luciang added a commit to luciang/fbzmq that referenced this pull request Apr 30, 2022
…age (#76610)

Summary:
X-link: pytorch/pytorch#76610

X-link: facebookincubator/gloo#328

Pull Request resolved: facebookarchive#38

X-link: facebook/CacheLib#137

Fix
  error: anonymous non-C-compatible type given name for linkage purposes by alias declaration; add a tag name here [-Werror,-Wnon-c-typedef-for-linkage]

Reviewed By: philippv

Differential Revision: D36043476

fbshipit-source-id: 953e43613479fb142ea2ccf29f3d0a39402e4122
luciang added a commit to luciang/pytorch that referenced this pull request Apr 30, 2022
…age (pytorch#76610)

Summary:
Pull Request resolved: pytorch#76610

X-link: facebookincubator/gloo#328

X-link: facebookarchive/fbzmq#38

X-link: facebook/CacheLib#137

Fix
  error: anonymous non-C-compatible type given name for linkage purposes by alias declaration; add a tag name here [-Werror,-Wnon-c-typedef-for-linkage]

Test Plan:
```name=before
buck-out/v2/gen/fbcode/d839c731f5505c62/datasec/messaging/auth/utils/__MessagingAuthUtils__/headers/datasec/messaging/auth/utils/MessagingAuthLogger.h:83:25: error: anonymous non-C-compatible type given name for linkage purposes by alias declaration; add a tag name here [-Werror,-Wnon-c-typedef-for-linkage]
  using LogInfo = struct {
```

CI

Reviewed By: philippv

Differential Revision: D36043476

fbshipit-source-id: 679f3b5efd0d233f6cb472e9cba54747ffd49c4e
facebook-github-bot pushed a commit to facebookarchive/fbzmq that referenced this pull request Apr 30, 2022
…age (#137)

Summary:
X-link: facebook/CacheLib#137

Fix
  error: anonymous non-C-compatible type given name for linkage purposes by alias declaration; add a tag name here [-Werror,-Wnon-c-typedef-for-linkage]

Reviewed By: philippv

Differential Revision: D36043476

fbshipit-source-id: df3fa684c8655184dd8f87045542cd7f5704a65e
facebook-github-bot pushed a commit to facebookincubator/gloo that referenced this pull request Apr 30, 2022
…age (#137)

Summary:
X-link: facebook/CacheLib#137

Fix
  error: anonymous non-C-compatible type given name for linkage purposes by alias declaration; add a tag name here [-Werror,-Wnon-c-typedef-for-linkage]

Reviewed By: philippv

Differential Revision: D36043476

fbshipit-source-id: df3fa684c8655184dd8f87045542cd7f5704a65e
facebook-github-bot pushed a commit to pytorch/pytorch that referenced this pull request Apr 30, 2022
…age (#137)

Summary:
X-link: facebook/CacheLib#137

Fix
  error: anonymous non-C-compatible type given name for linkage purposes by alias declaration; add a tag name here [-Werror,-Wnon-c-typedef-for-linkage]

Test Plan:
```name=before
buck-out/v2/gen/fbcode/d839c731f5505c62/datasec/messaging/auth/utils/__MessagingAuthUtils__/headers/datasec/messaging/auth/utils/MessagingAuthLogger.h:83:25: error: anonymous non-C-compatible type given name for linkage purposes by alias declaration; add a tag name here [-Werror,-Wnon-c-typedef-for-linkage]
  using LogInfo = struct {
```

CI

Reviewed By: philippv

Differential Revision: D36043476

fbshipit-source-id: df3fa684c8655184dd8f87045542cd7f5704a65e
pytorchmergebot pushed a commit to pytorch/pytorch that referenced this pull request May 2, 2022
…age (#137)

Summary:
X-link: facebook/CacheLib#137

Fix
  error: anonymous non-C-compatible type given name for linkage purposes by alias declaration; add a tag name here [-Werror,-Wnon-c-typedef-for-linkage]

Test Plan:
```name=before
buck-out/v2/gen/fbcode/d839c731f5505c62/datasec/messaging/auth/utils/__MessagingAuthUtils__/headers/datasec/messaging/auth/utils/MessagingAuthLogger.h:83:25: error: anonymous non-C-compatible type given name for linkage purposes by alias declaration; add a tag name here [-Werror,-Wnon-c-typedef-for-linkage]
  using LogInfo = struct {
```

CI

Reviewed By: philippv

Differential Revision: D36043476

Pull Request resolved: #76610
Approved by: https://github.com/philippv, https://github.com/osalpekar
igchor pushed a commit to igchor/CacheLib-1 that referenced this pull request May 16, 2022
…age (facebook#137)

Summary:
Pull Request resolved: facebook#137

Fix
  error: anonymous non-C-compatible type given name for linkage purposes by alias declaration; add a tag name here [-Werror,-Wnon-c-typedef-for-linkage]

Reviewed By: philippv

Differential Revision: D36043476

fbshipit-source-id: df3fa684c8655184dd8f87045542cd7f5704a65e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants