diff options
| author | Dr. David Alan Gilbert <dave@treblig.org> | 2020-11-20 03:14:58 +0000 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2020-11-22 11:53:01 +0100 |
| commit | 553cbea8f66abc08be9ebb9638b8a17b7ecf37a2 (patch) | |
| tree | 361ec29570579f1b24e8a7b8d35f5c8ebd17b2af /oox | |
| parent | 430bfe8908551b9ef0b8aaf9162b90560d72aed3 (diff) | |
Fix ESCHER_Prop_pVertices/ESCHER_Prop_pSegmentInfo output
Commit f1ec5dcfca45e reworked the output code to:
OString pathString = aPath.makeStringAndClear();
if ( !aPath.isEmpty() && pathString != "xe" )
Unfortunately the 'AndClear' means that the isEmpty is always true
so we never get the path string. This happens on complex polygons;
e.g. the orange arrows in tdf#138374
Change-Id: I41e69f2e5eb73b49ff512d2c1ca94423bc0bb202
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/106212
Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'oox')
| -rw-r--r-- | oox/source/export/vmlexport.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/oox/source/export/vmlexport.cxx b/oox/source/export/vmlexport.cxx index e3ed8547aaed..c708be927ef3 100644 --- a/oox/source/export/vmlexport.cxx +++ b/oox/source/export/vmlexport.cxx @@ -651,7 +651,7 @@ void VMLExport::Commit( EscherPropertyContainer& rProps, const tools::Rectangle& } } OString pathString = aPath.makeStringAndClear(); - if ( !aPath.isEmpty() && pathString != "xe" ) + if ( !pathString.isEmpty() && pathString != "xe" ) m_pShapeAttrList->add( XML_path, pathString ); } else |