Subject: Re: [patch 12/13] fb: Move and rename extern declaration for global_mode_option

From: Andrew Morton <[email-masked]>
Date: Mon, 15 Oct 2007 14:43:40 -0700

On Fri, 12 Oct 2007 16:51:04 +0200
Geert Uytterhoeven <[email-masked]> wrote:

> -    if (mode_option || (mode_option = global_mode_option)) {
> +    if (mode_option || (mode_option = fb_mode_option)) {

I guess that equals-which-looks-like-it-should-be-equals-equals really
is intended to be an assignment?

I guess.  After staring at it for a while.  It's a pretty obnoxious way
of coding it, especially in kernel context where such things are unexpected...