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) #328

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

luciang
Copy link
Contributor

@luciang luciang commented Apr 30, 2022

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

@facebook-github-bot
Copy link

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

…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/CacheLib that referenced this pull request Apr 30, 2022
…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

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 (#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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants