On 11/17/2014 08:34 AM, David Maciejak wrote:
> Are you sure it's safe to change the test in that following code ?
>
> -       if (shortenTitle && wPreferences.miniwin_title_balloon) {
> +       if (shortenTitle != NULL) {
>
>

It looks like the wPreferences.miniwin_title_balloon check is redundant 
now because of the following change:

>          if (wPreferences.miniwin_title_balloon) {
> -               shortenTitle = ShrinkString(font, title, width - 
> APERCU_BORDER);
> +               shortenTitle = ShrinkString(font, title, width - 
> APERCU_BORDER * 2);
>                  titleHeight = countLines(shortenTitle) * WMFontHeight(font) 
> + 4;
>                  height += titleHeight;
> +       } else {
> +               shortenTitle = NULL;
> +               titleHeight = 0;
>          }

Doug

Reply via email to