Subject: Re: bug#12463: 24.2; pos-visible-in-window-p gets slower over time

From: Eli Zaretskii <[email-masked]>
Date: Sat, 22 Sep 2012 11:36:38 +0300

> From: Chong Yidong <[email-masked]>
> Cc: Juanma Barranquero <[email-masked]>,  [email-masked],  [email-masked]
> Date: Sat, 22 Sep 2012 09:23:37 +0800
> 
> ! #ifdef HAVE_NTGUI
> !       /* If we failed to load the library before, don't try again.  */
> !       Lisp_Object tested = Fassq (target_type, Vlibrary_cache);
> !       if (CONSP (tested) && NILP (XCDR (tested)))
> ! 	type_valid = 0;
> !       else
> ! #endif
> ! 	{
> ! 	  /* If the load failed, avoid trying again.  */
> ! 	  type_valid = type->init (libraries);
> ! 	  CACHE_IMAGE_TYPE (target_type, type_valid ? Qt : Qnil);
> ! 	}
> !     }

The last comment seems to be redundant, given the one before it.

> ***************
> *** 8884,8892 ****
>     DEFSYM (Qxbm, "xbm");
>     ADD_IMAGE_TYPE (Qxbm);
>   
> -   define_image_type (&xbm_type, 1);
> -   define_image_type (&pbm_type, 1);

Why are these two lines being removed, while the corresponding
ADD_IMAGE_TYPE lines are not?

Thanks.