Re: [PATCH 3/3] move wifi activation fails logging to proper block

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Nov 11, 2011 at 01:38:14PM +0100, Vratislav Podzimek wrote:
> Otherwise we have 'wifi activation failed' log message even if there's no wifi-related line in kickstart.
> ---
>  loader/kickstart.c |    5 ++---
>  1 files changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/loader/kickstart.c b/loader/kickstart.c
> index b038406..12082b1 100644
> --- a/loader/kickstart.c
> +++ b/loader/kickstart.c
> @@ -671,11 +671,10 @@ int process_kickstart_wifi (struct loaderData_s * loaderData) {
>                      WIFI_PROTECTION_UNPROTECTED, NULL, loaderData->ipinfo_set, loaderData->ipv4,
>                      loaderData->gateway, loaderData->dns, loaderData->netmask);
>          }
> +        if (rc == WIFI_ACTIVATION_OK) loaderData->netDev_set = 1;
> +        else logMessage(ERROR, "wifi activation in kickstart failed");

Please split the if/else into separate lines, it makes it more readable.

Other than that, these look ok to me.

-- 
Brian C. Lane | Anaconda Team | IRC: bcl #anaconda | Port Orchard, WA (PST8PDT)

Attachment: pgp3DmIABQVjR.pgp
Description: PGP signature

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list

[Index of Archives]     [Kickstart]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [Yosemite Photos]     [KDE Users]     [Fedora Tools]
  Powered by Linux