{"action":"opened","issue":{"url":"https://api.github.com/repos/lwfinger/rtlwifi_new/issues/143","repository_url":"https://api.github.com/repos/lwfinger/rtlwifi_new","labels_url":"https://api.github.com/repos/lwfinger/rtlwifi_new/issues/143/labels{/name}","comments_url":"https://api.github.com/repos/lwfinger/rtlwifi_new/issues/143/comments","events_url":"https://api.github.com/repos/lwfinger/rtlwifi_new/issues/143/events","html_url":"https://github.com/lwfinger/rtlwifi_new/issues/143","id":172308496,"number":143,"title":"Possible Errors in code","user":{"login":"jshand2015","id":10791292,"avatar_url":"https://avatars.githubusercontent.com/u/10791292?v=3","gravatar_id":"","url":"https://api.github.com/users/jshand2015","html_url":"https://github.com/jshand2015","followers_url":"https://api.github.com/users/jshand2015/followers","following_url":"https://api.github.com/users/jshand2015/following{/other_user}","gists_url":"https://api.github.com/users/jshand2015/gists{/gist_id}","starred_url":"https://api.github.com/users/jshand2015/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jshand2015/subscriptions","organizations_url":"https://api.github.com/users/jshand2015/orgs","repos_url":"https://api.github.com/users/jshand2015/repos","events_url":"https://api.github.com/users/jshand2015/events{/privacy}","received_events_url":"https://api.github.com/users/jshand2015/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2016-08-21T07:08:25Z","updated_at":"2016-08-21T07:08:25Z","closed_at":null,"body":"please be aware that these below code issues do not interfere with the build process\r\n\r\n/home/jshand/rtlwifi/rtl8192ee/hw.c: In function ‘rtl92ee_set_hw_reg’:\r\n/home/jshand/rtlwifi/rtl8192ee/hw.c:538:4: warning: this ‘if’ clause does not guard... [-Wmisleading-indentation]\r\n    if (fac > 0xf)\r\n    ^~\r\n/home/jshand/rtlwifi/rtl8192ee/hw.c:540:5: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the ‘if’\r\n     for (i = 0; i < 4; i++) {\r\n     ^~~\r\n/home/jshand/rtlwifi/rtl8192ee/hw.c:544:6: warning: this ‘if’ clause does not guard... [-Wmisleading-indentation]\r\n      if ((reg[i] & 0x0f) > fac)\r\n      ^~\r\n/home/jshand/rtlwifi/rtl8192ee/hw.c:546:7: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the ‘if’\r\n       rtl_write_byte(rtlpriv,\r\n       ^~~~~~~~~~~~~~\r\n/home/jshand/rtlwifi/rtl8192ee/hw.c: In function ‘_rtl92ee_read_adapter_info’:\r\n/home/jshand/rtlwifi/rtl8192ee/hw.c:2138:14: warning: logical not is only applied to the left hand side of comparison [-Wlogical-not-parentheses]\r\n  if (!hwinfo == true)\r\n              ^~\r\n\r\n/home/jshand/rtlwifi/rtl8723ae/hw.c: In function ‘_rtl8723e_read_adapter_info’:\r\n/home/jshand/rtlwifi/rtl8723ae/hw.c:1635:14: warning: logical not is only applied to the left hand side of comparison [-Wlogical-not-parentheses]\r\n  if (!hwinfo == true)\r\n              ^~\r\n\r\n/home/jshand/rtlwifi/rtl8821ae/dm.c: In function ‘rtl8821ae_dm_txpower_tracking_callback_thermalmeter’:\r\n/home/jshand/rtlwifi/rtl8821ae/dm.c:2764:3: warning: this ‘for’ clause does not guard... [-Wmisleading-indentation]\r\n   for (p = RF90_PATH_A; p < MAX_PATH_NUM_8821A; p++)\r\n   ^~~\r\n/home/jshand/rtlwifi/rtl8821ae/dm.c:2767:4: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the ‘for’\r\n    RT_TRACE(rtlpriv, COMP_POWER_TRACKING, DBG_LOUD,\r\n    ^~~~~~~~"}}
{"actor":{"display_login":"jshand2015"}}