wxDatabase icon indicating copy to clipboard operation
wxDatabase copied to clipboard

Build warnings with mySQL

Open Jarod42 opened this issue 1 year ago • 3 comments

From CI: https://github.com/Jarod42/wxDatabase/actions/runs/11632894931/job/32396938912

/Users/runner/work/wxDatabase/wxDatabase/src/database/mysql/mysql_preparedStatement_resultset.cpp:134:37: warning: equality comparison with extraneous parentheses [-Wparentheses-equality]
    if ((*(pResultBinding->is_null) == false))
         ~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~
/Users/runner/work/wxDatabase/wxDatabase/src/database/mysql/mysql_preparedStatement_resultset.cpp:134:37: note: remove extraneous parentheses around the comparison to silence this warning
    if ((*(pResultBinding->is_null) == false))
        ~                           ^       ~
/Users/runner/work/wxDatabase/wxDatabase/src/database/mysql/mysql_preparedStatement_resultset.cpp:134:37: note: use '=' to turn this equality comparison into an assignment
    if ((*(pResultBinding->is_null) == false))
                                    ^~
                                    =
/Users/runner/work/wxDatabase/wxDatabase/src/database/mysql/mysql_preparedStatement_resultset.cpp:148:37: warning: equality comparison with extraneous parentheses [-Wparentheses-equality]
    if ((*(pResultBinding->is_null) == false))
         ~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~
/Users/runner/work/wxDatabase/wxDatabase/src/database/mysql/mysql_preparedStatement_resultset.cpp:148:37: note: remove extraneous parentheses around the comparison to silence this warning
    if ((*(pResultBinding->is_null) == false))
        ~                           ^       ~
/Users/runner/work/wxDatabase/wxDatabase/src/database/mysql/mysql_preparedStatement_resultset.cpp:148:37: note: use '=' to turn this equality comparison into an assignment
    if ((*(pResultBinding->is_null) == false))
                                    ^~
                                    =
/Users/runner/work/wxDatabase/wxDatabase/src/database/mysql/mysql_preparedStatement_resultset.cpp:1[90](https://github.com/Jarod42/wxDatabase/actions/runs/11632894931/job/32396938912#step:6:91):37: warning: equality comparison with extraneous parentheses [-Wparentheses-equality]
    if ((*(pResultBinding->is_null) == false))
         ~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~
/Users/runner/work/wxDatabase/wxDatabase/src/database/mysql/mysql_preparedStatement_resultset.cpp:190:37: note: remove extraneous parentheses around the comparison to silence this warning
    if ((*(pResultBinding->is_null) == false))
        ~                           ^       ~
/Users/runner/work/wxDatabase/wxDatabase/src/database/mysql/mysql_preparedStatement_resultset.cpp:190:37: note: use '=' to turn this equality comparison into an assignment
    if ((*(pResultBinding->is_null) == false))
                                    ^~
                                    =
/Users/runner/work/wxDatabase/wxDatabase/src/database/mysql/mysql_preparedStatement_resultset.cpp:202:37: warning: equality comparison with extraneous parentheses [-Wparentheses-equality]
    if ((*(pResultBinding->is_null) == false))
         ~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~
/Users/runner/work/wxDatabase/wxDatabase/src/database/mysql/mysql_preparedStatement_resultset.cpp:202:37: note: remove extraneous parentheses around the comparison to silence this warning
    if ((*(pResultBinding->is_null) == false))
        ~                           ^       ~
/Users/runner/work/wxDatabase/wxDatabase/src/database/mysql/mysql_preparedStatement_resultset.cpp:202:37: note: use '=' to turn this equality comparison into an assignment
    if ((*(pResultBinding->is_null) == false))
                                    ^~
                                    =
/Users/runner/work/wxDatabase/wxDatabase/src/database/mysql/mysql_preparedStatement_resultset.cpp:217:37: warning: equality comparison with extraneous parentheses [-Wparentheses-equality]
    if ((*(pResultBinding->is_null) == false))
         ~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~
/Users/runner/work/wxDatabase/wxDatabase/src/database/mysql/mysql_preparedStatement_resultset.cpp:217:37: note: remove extraneous parentheses around the comparison to silence this warning
    if ((*(pResultBinding->is_null) == false))
        ~                           ^       ~
/Users/runner/work/wxDatabase/wxDatabase/src/database/mysql/mysql_preparedStatement_resultset.cpp:217:37: note: use '=' to turn this equality comparison into an assignment
    if ((*(pResultBinding->is_null) == false))
                                    ^~
                                    =
/Users/runner/work/wxDatabase/wxDatabase/src/database/mysql/mysql_preparedStatement_resultset.cpp:260:37: warning: equality comparison with extraneous parentheses [-Wparentheses-equality]
    if ((*(pResultBinding->is_null) == false))
         ~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~
/Users/runner/work/wxDatabase/wxDatabase/src/database/mysql/mysql_preparedStatement_resultset.cpp:260:37: note: remove extraneous parentheses around the comparison to silence this warning
    if ((*(pResultBinding->is_null) == false))
        ~                           ^       ~
/Users/runner/work/wxDatabase/wxDatabase/src/database/mysql/mysql_preparedStatement_resultset.cpp:260:37: note: use '=' to turn this equality comparison into an assignment
    if ((*(pResultBinding->is_null) == false))
                                    ^~
                                    =
In file included from /Users/runner/work/wxDatabase/wxDatabase/src/database/mysql/mysql_resultset_metadata.cpp:5:
/opt/homebrew/include/mysql/mysql_com.h:3:2: warning: This file should not be included by clients, include only <mysql.h> [-W#warnings]
#warning This file should not be included by clients, include only <mysql.h>
 ^
1 warning generated.
6 warnings generated.

Jarod42 avatar Nov 03 '24 11:11 Jarod42

Fixed by https://github.com/Jarod42/wxDatabase/tree/warnings Waiting previous PR about CI before to submit this one.

Jarod42 avatar Nov 03 '24 11:11 Jarod42

Thanks, I have been having a tough week, but have not forgotten this one and the other one. I hope to come to it over the weekend and fix it!

mtangoo avatar Nov 15 '24 05:11 mtangoo

No urgency on my side. Several changes would be incoming in long term :-)

Jarod42 avatar Nov 15 '24 10:11 Jarod42