unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Problem report #115: base/src/emacs/src/font.c (font_update_lface); UNINIT
@ 2008-12-02 22:24 Dan Nicolaescu
  2008-12-03  5:17 ` Chong Yidong
  0 siblings, 1 reply; 2+ messages in thread
From: Dan Nicolaescu @ 2008-12-02 22:24 UTC (permalink / raw)
  To: emacs-devel

CID: 115
Checker: UNINIT (help)
File: base/src/emacs/src/font.c
Function: font_update_lface
Description: Using uninitialized value "point"

3045 	    attrs[LFACE_SWIDTH_INDEX] = FONT_WIDTH_FOR_FACE (spec);
3046 	  if (! NILP (AREF (spec, FONT_SIZE_INDEX)))
3047 	    {

Event var_decl: Declared variable "point" without initializer
Also see events: [uninit_use]

3048 	      int point;
3049 	

At conditional (1): "((0), (spec & -8))->contents[8] & 7 == 0" taking false path

3050 	      if (INTEGERP (AREF (spec, FONT_SIZE_INDEX)))
3051 		{
3052 		  Lisp_Object val;
3053 		  int dpi = f->resy;
3054 	
3055 		  val = Ffont_get (spec, QCdpi);
3056 		  if (! NILP (val))
3057 		    dpi = XINT (val);
3058 		  point = PIXEL_TO_POINT (XINT (AREF (spec, FONT_SIZE_INDEX)) * 10,
3059 					  dpi);
3060 		}

At conditional (2): "((0), (spec & -8))->contents[8] & 7 == 6" taking false path

3061 	      else if (FLOATP (AREF (spec, FONT_SIZE_INDEX)))
3062 		point = XFLOAT_DATA (AREF (spec, FONT_SIZE_INDEX)) * 10;

Event uninit_use: Using uninitialized value "point"
Also see events: [var_decl]

3063 	      attrs[LFACE_HEIGHT_INDEX] = make_number (point);
3064 	    }
3065 	}
3066 	




^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: Problem report #115: base/src/emacs/src/font.c (font_update_lface); UNINIT
  2008-12-02 22:24 Problem report #115: base/src/emacs/src/font.c (font_update_lface); UNINIT Dan Nicolaescu
@ 2008-12-03  5:17 ` Chong Yidong
  0 siblings, 0 replies; 2+ messages in thread
From: Chong Yidong @ 2008-12-03  5:17 UTC (permalink / raw)
  To: Dan Nicolaescu; +Cc: emacs-devel

Dan Nicolaescu <dann@ics.uci.edu> writes:

> CID: 115
> Checker: UNINIT (help)
> File: base/src/emacs/src/font.c
> Function: font_update_lface
> Description: Using uninitialized value "point"

Fixed, thanks.




^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2008-12-03  5:17 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-12-02 22:24 Problem report #115: base/src/emacs/src/font.c (font_update_lface); UNINIT Dan Nicolaescu
2008-12-03  5:17 ` Chong Yidong

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).