emacs-devel
[Top][All Lists]
Advanced

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

Re: compilation warning


From: Christoph
Subject: Re: compilation warning
Date: Mon, 28 Jun 2010 19:38:10 -0600
User-agent: Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.2.4) Gecko/20100608 Thunderbird/3.1

On 6/28/2010 6:55 PM, Dan Nicolaescu wrote:

Compiling with -Wempty-body gives this warning:

emacs/src/frame.c:1390: warning: suggest braces around empty body in an 'if' 
statement

The code in question is:

   if (NILP (Vrun_hooks) || !NILP (Fframe_parameter (frame, intern 
("tooltip"))))
     ;

It looks a bit odd.  What's the right fix here?



Something like this:

if (NILP (Vrun_hooks) || !NILP (Fframe_parameter (frame, intern ("tooltip"))))
  {
    ; /* empty body */
  }

Christoph



reply via email to

[Prev in Thread] Current Thread [Next in Thread]