Project

General

Profile

Actions
Copy link

Bug #13649

closed

stackalign: unused variables

Added by Toomas Soome about 1 year ago. Updated about 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Toomas Soome
Category:
tests
Start date:
Due date:
% Done:

100%

Estimated time:
Difficulty:
Bite-size
Tags:
Gerrit CR:
1353

Description

Build errors on SPARC:

    stackalign.c: In function 'main':
    stackalign.c:96:15: error: unused variable 'rc' [-Werror=unused-variable]
       96 |  int door_fd, rc;
          |               ^~
    stackalign.c:96:6: error: unused variable 'door_fd' [-Werror=unused-variable]
       96 |  int door_fd, rc;
          |      ^~~~~~~
    stackalign.c:95:12: error: unused variable 'tid' [-Werror=unused-variable]
       95 |  pthread_t tid;
          |            ^~~
    stackalign.c:94:19: error: unused variable 'uc' [-Werror=unused-variable]
       94 |  ucontext_t back, uc;
          |                   ^~
    stackalign.c:94:13: error: unused variable 'back' [-Werror=unused-variable]
       94 |  ucontext_t back, uc;
          |             ^~~~
    stackalign.c:90:13: error: unused variable 'arg' [-Werror=unused-variable]
       90 |  test_ctx_t arg = {
          |             ^~~
    stackalign.c:86:13: error: unused variable 'da' [-Werror=unused-variable]
       86 |  door_arg_t da = {
          |             ^~
    cc1: all warnings being treated as errors

Testing done: build is clean (on both sparc and x86).

Actions
Copy link
 #1

Updated by Electric Monk about 1 year ago

  • Gerrit CR set to 1353
Actions
Copy link
 #2

Updated by Toomas Soome about 1 year ago

  • Description updated (diff)
Actions
Copy link
 #3

Updated by Electric Monk about 1 year ago

  • Status changed from In Progress to Closed
  • % Done changed from 90 to 100

git commit c96f32ae7ef73b9db2b9b81e2e1e2b206795d024

commit  c96f32ae7ef73b9db2b9b81e2e1e2b206795d024
Author: Toomas Soome <tsoome@me.com>
Date:   2021-03-19T18:59:42.000Z

    13649 stackalign: unused variables
    Reviewed by: Andy Fiddaman <andy@omnios.org>
    Reviewed by: Andrew Stormont <andyjstormont@gmail.com>
    Approved by: Robert Mustacchi <rm@fingolfin.org>

Actions
Copy link

Also available in: Atom PDF