C参数设置但未使用

时间:2016-02-10 20:11:23

标签: c parameter-passing

我正在使用C来制作鹅卵石表盘。我收到了错误parameter 'light_bitmap' set but not used [-Werror=unused-but-set-parameter]。然而,该变量在我的函数中使用。我不确定这条消息是什么意思。以下是我的代码:

在标题声明中:

void backlight_subscribe(GBitmap *light_bitmap, int start_hour, int end_hour);

在定义中.c:

void backlight_subscribe(GBitmap *light_bitmap, int start_hour, int end_hour) {
    int hour = get_current_hour();
    if((hour >= start_hour) || (hour < end_hour)) {
        accel_tap_service_subscribe(tap_handler);
        light_bitmap = gbitmap_create_with_resource(RESOURCE_ID_IMAGE_LIGHT_ON_ICON);
    } else {
        accel_tap_service_unsubscribe();
        light_bitmap = gbitmap_create_with_resource(RESOURCE_ID_IMAGE_LIGHT_OFF_ICON);
    }
}

并在main.c中:

static GBitmap *light_bitmap;
...
static void window_load(Window *window) {
     ...
     backlight_subscribe(light_bitmap, 22, 8);
     ...
}

static void init(void) {
    s_main_window = window_create();
    window_set_window_handlers(s_main_window, (WindowHandlers) {
        .load = window_load,
        .unload = window_unload,
    });
    window_stack_push(s_main_window, false); 
}

...

int main(void) {
    init();
    app_event_loop();
    deinit();
}

完整的错误消息:

[16/28] c: src/accel_sensor.c -> build/src/accel_sensor.c.16.o
../src/accel_sensor.c: In function 'backlight_subscribe':
../src/accel_sensor.c:22:35: error: parameter 'light_bitmap' set but not used [-Werror=unused-but-set-parameter]
 void backlight_subscribe(GBitmap *light_bitmap, int start_hour, int end_hour) {
                                   ^
cc1: all warnings being treated as errors
Waf: Leaving directory `/home/adminuser/pebble/pebble-app/watchface/build'
Build failed

可能有人看到我做错了什么?

1 个答案:

答案 0 :(得分:3)

正如错误所述,在您的backlight_subscribe功能中,您为light_bitmap分配了一个值,但您并未从中读取该值。这意味着它没有被使用。

在C中,所有参数都按值传递。因此,您分配给light_bitmap的值仅在该函数中可见。如果您打算更改要传入的变量,则需要传入相关变量的地址,更改函数以接受指向该变量的指针,然后在分配时取消引用指针它:

void backlight_subscribe(GBitmap **light_bitmap, int start_hour, int end_hour) {
    int hour = get_current_hour();
    if((hour >= start_hour) || (hour < end_hour)) {
        accel_tap_service_subscribe(tap_handler);
        *light_bitmap = gbitmap_create_with_resource(RESOURCE_ID_IMAGE_LIGHT_ON_ICON);
    } else {
        accel_tap_service_unsubscribe();
        *light_bitmap = gbitmap_create_with_resource(RESOURCE_ID_IMAGE_LIGHT_OFF_ICON);
    }
}

然后你调用这样的函数:

backlight_subscribe(&light_bitmap, 22, 8);

作为旁注,函数参数light_bitmap正在屏蔽同名的全局变量。这可能会引起混淆,所以尽量避免这种情况。