svn commit: r327037 - head/sys/dev/acpica

Cy Schubert Cy.Schubert at komquats.com
Wed Dec 20 20:33:50 UTC 2017


In message <201712201914.vBKJEG68087359 at repo.freebsd.org>, Warner Losh 
writes:
> Author: imp
> Date: Wed Dec 20 19:14:16 2017
> New Revision: 327037
> URL: https://svnweb.freebsd.org/changeset/base/327037
>
> Log:
>   Simplify the code a bit.
>   
>   Replace clumsy for(;;) { if (foo) break; ...} with simpler
>   while (!foo) { ... }.
>   
>   Sponsored by: Netflix
>   Differential Revision: https://reviews.freebsd.org/D13546
>
> Modified:
>   head/sys/dev/acpica/acpi.c
>
> Modified: head/sys/dev/acpica/acpi.c
> =============================================================================
> =
> --- head/sys/dev/acpica/acpi.c	Wed Dec 20 19:14:11 2017	(r32703
> 6)
> +++ head/sys/dev/acpica/acpi.c	Wed Dec 20 19:14:16 2017	(r32703
> 7)
> @@ -1014,10 +1014,7 @@ acpi_hint_device_unit(device_t acdev, device_t child, 
>       * name to see if one's resources are a subset of this device.
>       */
>      line = 0;
> -    for (;;) {
> -	if (resource_find_dev(&line, name, &unit, "at", NULL) != 0)
> -	    break;
> -
> +    while (resource_find_dev(&line, name, &unit, "at", NULL) == 0) {
>  	/* Must have an "at" for acpi or isa. */
>  	resource_string_value(name, unit, "at", &s);
>  	if (!(strcmp(s, "acpi0") == 0 || strcmp(s, "acpi") == 0 ||
>


Would it make sense we upline this to reduce future MFV diffs?


-- 
Cheers,
Cy Schubert <Cy.Schubert at cschubert.com>
FreeBSD UNIX:  <cy at FreeBSD.org>   Web:  http://www.FreeBSD.org

	The need of the many outweighs the greed of the few.




More information about the svn-src-all mailing list