Lars Gullik Bjønnes wrote:
> I belive this one is safe. (I'll commit after work tomorrow(today))
> Index: src/text2.C
> ===================================================================
> --- src/text2.C       (revision 14329)
> +++ src/text2.C       (working copy)
> @@ -212,7 +212,7 @@
>                       break;
>               }
>       // Realize against environment font information
> -     if (pit < pars_.size())
> +     if (pit < pit_type(pars_.size()))
>               font.realize(outerFont(pit, pars_));
>  
>       // Realize with the fonts of lesser depth.

Isn't it better to fix pit_type?

type.h:

        /// a type for paragraph offsets
        // FIXME: should be unsigned as well.
        // however, simply changing it breaks a downward loop somewhere...
        typedef ptrdiff_t  pit_type;

When you remove the warning we maybe will forget about it,
so I would not remove this warning.

Peter

Reply via email to