SWAP-DA: remove invalid semicolons on energy module
authorSeung-Woo Kim <sw0312.kim@samsung.com>
Sat, 21 Jan 2017 03:26:22 +0000 (12:26 +0900)
committerJaechul Lee <jcsing.lee@samsung.com>
Tue, 14 Feb 2017 04:26:24 +0000 (13:26 +0900)
There are invalid semicolons after if clause on energy module and
it causes following build error with gcc 6, so this patch removes
the invalid semicolons.

  kernel/swap/energy/energy.c: In function 'calc_wifi_recv_energy':
  kernel/swap/energy/energy.c:640:2: error: this 'if' clause does not guard... [-Werror=misleading-indentation]
    if (len <= 0 || check_wlan0(sock));
    ^~
  kernel/swap/energy/energy.c:641:3: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the 'if'
     return;
     ^~~~~~
  kernel/swap/energy/energy.c: In function 'calc_wifi_send_energy':
  kernel/swap/energy/energy.c:653:2: error: this 'if' clause does not guard... [-Werror=misleading-indentation]
    if (len <= 0 || check_wlan0(sock));
    ^~
  kernel/swap/energy/energy.c:654:3: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the 'if'
     return;
     ^~~~~~

Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>
kernel/swap/energy/energy.c

index 2cfd48a1126179e8ff44ad2b83e18dbca3a299b3..02914d2b6194d7ed32b951516477ec84ede1a518 100644 (file)
@@ -637,7 +637,7 @@ static void calc_wifi_recv_energy(struct socket *sock, int len)
 {
        struct energy_data *ed;
 
-       if (len <= 0 || check_wlan0(sock));
+       if (len <= 0 || check_wlan0(sock))
                return;
 
        ed = get_energy_data_by_socket(current, sock);
@@ -650,7 +650,7 @@ static void calc_wifi_send_energy(struct socket *sock, int len)
 {
        struct energy_data *ed;
 
-       if (len <= 0 || check_wlan0(sock));
+       if (len <= 0 || check_wlan0(sock))
                return;
 
        ed = get_energy_data_by_socket(current, sock);