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

gdb/xcoffread: Remove unused extra_lines variable #59

Closed
wants to merge 1 commit into from

Conversation

a4lg
Copy link
Owner

@a4lg a4lg commented Sep 25, 2022

@a4lg a4lg force-pushed the clang-nowarn-gdb-remove-unused-1 branch 12 times, most recently from 74c651d to 000b68b Compare September 28, 2022 05:27
Clang generates a warning if there is a variable that is set but not used
otherwise ("-Wunused-but-set-variable").  On the default configuration, it
causes a build failure (unless "--disable-werror" is specified).

The only extra_lines use in arrange_linetable function is removed on the
commit 558802e
("gdb: change subfile::line_vector to an std::vector").  So, this variable
should be removed to prevent a build failure.
@a4lg a4lg force-pushed the clang-nowarn-gdb-remove-unused-1 branch from 000b68b to 44d5ccb Compare September 28, 2022 07:21
@a4lg
Copy link
Owner Author

a4lg commented Sep 30, 2022

Closed because upstreamed as commit f370047.

@a4lg a4lg closed this Sep 30, 2022
@a4lg a4lg deleted the clang-nowarn-gdb-remove-unused-1 branch September 30, 2022 04:25
@a4lg a4lg added the bug Something isn't working label Oct 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant