Bug 142021 - [EMF] SetPolyfilMode Mode = 2 ("winding") is ignored
Summary: [EMF] SetPolyfilMode Mode = 2 ("winding") is ignored
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: graphics stack (show other bugs)
Version:
(earliest affected)
unspecified
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 84199 EMF-WMF 138147 143876
  Show dependency treegraph
 
Reported: 2021-05-01 15:41 UTC by Valek Filippov
Modified: 2024-03-25 03:13 UTC (History)
3 users (show)

See Also:
Crash report or crash signature:


Attachments
EMF sample (364 bytes, image/x-emf)
2021-05-01 15:43 UTC, Valek Filippov
Details
Screenshot of the sample file in MSO for win (2.11 KB, image/png)
2021-05-01 15:45 UTC, Valek Filippov
Details
EMF with PolyfillMode set to 0 beforr the bottom star record (364 bytes, image/x-emf)
2021-05-01 15:47 UTC, Valek Filippov
Details
EMF with PolyfillMode set to 6 before the bottom star record (364 bytes, image/x-emf)
2021-05-01 15:47 UTC, Valek Filippov
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Valek Filippov 2021-05-01 15:41:58 UTC
Description:
[MS-EMF] describes two values for PolygonFillMode enum:
Alternate = 1
Winding = 2

LO seems to ignore 'winding' mode and apply 'alternate'.

Steps to Reproduce:
Open EMF sample (star_pfm2.emf) in LO.

Actual Results:
Both stars rendered with unfilled pentagon in the center 

Expected Results:
Bottom star should be fully filled.


Reproducible: Always


User Profile Reset: No



Additional Info:
Implementation in MSO for Windows seems to consider anything other than 1 as "Winding".
Implementation in MS wordpad seems to consider anything other than 2 as "Alternate".

Because of the inconsistency in MS own implementation, decision is needed with what to be compatible.
Comment 1 Valek Filippov 2021-05-01 15:43:41 UTC
Created attachment 171570 [details]
EMF sample

PolyfillMode is set to 1 before top star record.

It set to 2 after that before the bottom start record.
Comment 2 Valek Filippov 2021-05-01 15:45:32 UTC
Created attachment 171571 [details]
Screenshot of the sample file in MSO for win
Comment 3 Valek Filippov 2021-05-01 15:47:09 UTC
Created attachment 171572 [details]
EMF with PolyfillMode set to 0 beforr the bottom star record

(for completeness, a file with undocumented value in SetPolyfillMode Mode)
Comment 4 Valek Filippov 2021-05-01 15:47:42 UTC
Created attachment 171573 [details]
EMF with PolyfillMode set to 6 before the bottom star record
Comment 6 Roman Kuznetsov 2021-05-03 13:41:49 UTC
confirm in

Version: 7.2.0.0.alpha0+ / LibreOffice Community
Build ID: a52590d76b89dc75be2aa87f4287624c89f1e82f
CPU threads: 4; OS: Mac OS X 11.3; UI render: default; VCL: osx
Locale: ru-RU (ru_RU.UTF-8); UI: en-US
Calc: threaded
Comment 7 Chris Sherlock 2021-10-09 23:48:50 UTC
Hossein, is this related to the work you are doing on winding?
Comment 8 Hossein 2021-10-10 10:35:41 UTC
(In reply to Chris Sherlock from comment #7)
> Hossein, is this related to the work you are doing on winding?
Chris: Yes. Please take a look at: tdf#84199. It depends on this issue.
Comment 9 QA Administrators 2024-03-25 03:13:25 UTC
Dear Valek Filippov,

To make sure we're focusing on the bugs that affect our users today, LibreOffice QA is asking bug reporters and confirmers to retest open, confirmed bugs which have not been touched for over a year.

There have been thousands of bug fixes and commits since anyone checked on this bug report. During that time, it's possible that the bug has been fixed, or the details of the problem have changed. We'd really appreciate your help in getting confirmation that the bug is still present.

If you have time, please do the following:

Test to see if the bug is still present with the latest version of LibreOffice from https://www.libreoffice.org/download/

If the bug is present, please leave a comment that includes the information from Help - About LibreOffice.
 
If the bug is NOT present, please set the bug's Status field to RESOLVED-WORKSFORME and leave a comment that includes the information from Help - About LibreOffice.

Please DO NOT

Update the version field
Reply via email (please reply directly on the bug tracker)
Set the bug's Status field to RESOLVED - FIXED (this status has a particular meaning that is not 
appropriate in this case)


If you want to do more to help you can test to see if your issue is a REGRESSION. To do so:
1. Download and install oldest version of LibreOffice (usually 3.3 unless your bug pertains to a feature added after 3.3) from https://downloadarchive.documentfoundation.org/libreoffice/old/

2. Test your bug
3. Leave a comment with your results.
4a. If the bug was present with 3.3 - set version to 'inherited from OOo';
4b. If the bug was not present in 3.3 - add 'regression' to keyword


Feel free to come ask questions or to say hello in our QA chat: https://web.libera.chat/?settings=#libreoffice-qa

Thank you for helping us make LibreOffice even better for everyone!

Warm Regards,
QA Team

MassPing-UntouchedBug