Words filter compiler error with GCC 4.7

[Matus Uzak]

Hi,

I recommend to use filters/libmso/pole.cpp in the starwriter and the
hancomword filter.  I can take care of the starwriter filter.


On Sat, Mar 24, 2012 at 3:30 AM, Yue Liu <yue.liu at mail.com> wrote:

> looks like it's okay to rename r with other name.
>
> 2012/3/24 Christoph Feck <christoph at maxiom.de>:
> > Hi,
> >
> > The calligra master builds fails with GCC 4.7 due to an error in the
> > "hancomwords" import filter.
> >
> > The loop in filters/words/hancomword/pole.cpp:869 reads:
> >
> >        for (unsigned r = 0; r < header->num_mbat; r++) {
> >            unsigned long r = loadBigBlock(mblock, buffer2, bbat-
> >>blockSize);
> >            if (r != bbat->blockSize) {
> >                delete[] buffer2;
> >                return;
> >            }
> >            for (unsigned s = 0; s < bbat->blockSize - 4; s += 4) {
> >                if (k >= header->num_bat) break;
> >                else  blocks[k++] = readU32(buffer2 + s);
> >            }
> >            mblock = readU32(buffer2 + bbat->blockSize - 4);
> >        }
> >
> > The error is that the variable "r" is used as the loop counter, as
> > well recomputed inside the loop. This clearly is a bug, but I do not
> > know the code to fix it.
> >
> > Christoph Feck (kdepepo)
> > KDE Quality Team
> > _______________________________________________
> > calligra-devel mailing list
> > calligra-devel at kde.org
> > https://mail.kde.org/mailman/listinfo/calligra-devel
>
>
>
> --
> Best Regards,
> Yue Liu
> _______________________________________________
> calligra-devel mailing list
> calligra-devel at kde.org
> https://mail.kde.org/mailman/listinfo/calligra-devel
>



-- 
Matus Uzak
Software Designer
Ixonos Slovakia s.r.o.
Sturova 27, 040 01 Kosice, Slovakia
mobile 0421 918 718 958
email: matus.uzak at ixonos.com
http://www.ixonos.com
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20120326/2d9448f9/attachment.htm>