使用clang编译openresty ,rds-csv-nginx-module模块报一个警告

wendal1985 · 2012-05-13T16:10:23+00:00

2012/5/13 agentzh <age...@gmail.com> Hi, folks! After two weeks' development, I'm happy to announce the new development release of ngx_ope...

使用clang编译openresty ,rds-csv-nginx-module模块报一个警告

wendal1985

使用默认配置:

./configure --with-cc=lang

系统: OpenSUSE 12.1 x64

警告信息如下

../rds-csv-nginx-module-0.05rc1/src/ngx_http_rds_csv_output.c:156:11: warning: _expression_ result unused [-Wunused-value]

+ ngx_get_num_size(header->affected_rows);

^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

1 warning generated.

--
Wendal Chen
GuangDong China

wendal1985

看了一下代码,原来是写错


    size += header->errstr.len + escape

          + ngx_get_num_size(header->insert_id);

          + ngx_get_num_size(header->affected_rows);




多了一个;号!!

2012/5/13 Wendal Chen <wend...@gmail.com>

使用默认配置:
./configure --with-cc=lang

系统: OpenSUSE 12.1 x64

警告信息如下

../rds-csv-nginx-module-0.05rc1/src/ngx_http_rds_csv_output.c:156:11: warning: _expression_ result unused [-Wunused-value]

+ ngx_get_num_size(header->affected_rows);
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 warning generated.

--
Wendal Chen
GuangDong China

--
Wendal Chen
GuangDong China

wendal1985

clang编译luajit 2.0.0beta9也会fail, 官方在beta10中已经修复这个问题

http://luajit.org/changes.html

Replace <unwind.h> definitions with our own.

2012/5/13 Wendal Chen <wend...@gmail.com>

看了一下代码,原来是写错
    size += header->errstr.len + escape



          + ngx_get_num_size(header->insert_id);



          + ngx_get_num_size(header->affected_rows);








多了一个;号!!
2012/5/13 Wendal Chen <wend...@gmail.com>
使用默认配置:
./configure --with-cc=lang

系统: OpenSUSE 12.1 x64

警告信息如下

../rds-csv-nginx-module-0.05rc1/src/ngx_http_rds_csv_output.c:156:11: warning: _expression_ result unused [-Wunused-value]

+ ngx_get_num_size(header->affected_rows);
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 warning generated.

--
Wendal Chen
GuangDong China

--
Wendal Chen
GuangDong China

--
Wendal Chen
GuangDong China

agentzh

2012/5/13 Wendal Chen <wend...@gmail.com>

clang编译luajit 2.0.0beta9也会fail, 官方在beta10中已经修复这个问题
http://luajit.org/changes.html
Replace <unwind.h> definitions with our own.

刚注意到 LuaJIT 2.0 beta10 都发布了！哈哈！好呀，我后面也该着手升级了 :)

Thanks!
-agentzh

agentzh

2012/5/13 Wendal Chen <wend...@gmail.com>
> 看了一下代码,原来是写错
>     size += header->errstr.len + escape
>           + ngx_get_num_size(header->insert_id);
>           + ngx_get_num_size(header->affected_rows);
> 多了一个;号!!
>

这个 bug 发现得好！我刚刚已提交了补丁修复了这个问题：

    https://github.com/agentzh/rds-csv-nginx-module/commit/c15d3a4

当 affected rows 字段的值大于一位数（比如两位数、三位数、等等）的时候，缓存区大小的计算便是错误的。

多谢报告！看来我也得多用 clang 编译我的东西 ;)

Best regards,
-agentzh