Issue 121066

Summary: Some warning message show up, when building sw/source/filter/ww8/wrtww8.cxx
Product: Writer Reporter: zhengfan <zheng.easyfan>
Component: codeAssignee: AOO issues mailing list <issues>
Status: ACCEPTED --- QA Contact:
Severity: Normal    
Priority: P3 CC: chengjh, issues, kschenk, pavel
Version: 4.0.0-dev   
Target Milestone: ---   
Hardware: All   
OS: All   
Issue Type: DEFECT Latest Confirmation in: ---
Developer Difficulty: ---
Attachments:
Description Flags
The patch for reivew zheng.easyfan: review?

Description zhengfan 2012-09-20 06:16:12 UTC
svn commit: r1386590 - in /incubator/ooo/trunk: ./ main/sw/inc/ main/sw/source/core/doc/ main/sw/source/core/tox/ main/sw/source/filter/ww8/ main/sw/source/filter/ww8/dump/ main/sw/source/ui/index/ test/ test/testcommon/source/org/openoffice/test

> +void WW8_WrtBookmarks::MoveFieldMarks(sal_uLong nFrom,sal_uLong nTo)
> +{
> +    std::pair<CPItr,CPItr> aRange = aSttCps.equal_range(nFrom);
> +    CPItr aItr = aRange.first;
> +    while (aItr != aRange.second)
> +    {
> +        if (aItr->second)
> +        {
> +            if (aItr->second->first == nFrom)

the last line is a source of compiler warning:

sw/source/filter/ww8/wrtww8.cxx: In member function ‘void WW8_WrtBookmarks::MoveFieldMarks(sal_uLong, sal_uLong)’:
sw/source/filter/ww8/wrtww8.cxx:315: warning: comparison between signed and unsigned integer expressions

Thanks Pavel Janík <Pavel@janik.cz> for reporting the issue.
Comment 1 zhengfan 2012-09-21 10:34:19 UTC
Created attachment 79613 [details]
The patch for reivew
Comment 2 pavel 2012-09-26 05:34:26 UTC
Compiles cleanly (the patch is missing new line at the end of the file though).
Comment 3 Marcus 2017-05-20 11:01:00 UTC
Reset assigne to the default "issues@openoffice.apache.org".