Apache OpenOffice (AOO) Bugzilla – Full Text Issue Listing |
Summary: | [From Symphony] The value of the "delay" option can not be kept if save the presentation to a .ppt doc via Symphony. | ||||||
---|---|---|---|---|---|---|---|
Product: | Impress | Reporter: | Steve Yin <steve.yin.aoo> | ||||
Component: | save-export | Assignee: | WangZhe <kingwisemmx> | ||||
Status: | CLOSED DUPLICATE | QA Contact: | |||||
Severity: | Normal | ||||||
Priority: | P3 | CC: | kingwisemmx, louqingle, sunying | ||||
Version: | 4.0.0-dev | ||||||
Target Milestone: | --- | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
Issue Type: | DEFECT | Latest Confirmation in: | --- | ||||
Developer Difficulty: | --- | ||||||
Attachments: |
|
Description
Steve Yin
2012-08-24 06:02:03 UTC
Created attachment 79120 [details]
Fix patch for this issue
Root Cause:
When saving the "Delay" value, the AnimationExporter::exportAnimEvent function use an uninitialized value eTiming.
Resolution:
Just initialize this value eTiming = Timing_INDEFINITE.
the issue 119711 has the same root cause with this issue. at line 1381 in file "\main\sd\source\filter\eppt\pptexanimations.cxx",the "if ( eTiming == Timing_INDEFINITE )" is always false, and the nBegin value never is assigned by fTiming. so the "Delay" value always is zero.\ if initialize this value eTiming = Timing_INDEFINITE, and the "if ( eTiming == Timing_INDEFINITE )" is always true, and the nBegin value alwalys is assigned by fTiming, so Is it necessary to judge eTiming when fTiming has a value? In my opinion,according in "\main\offapi\com\sun\star\animationstiming.idl", describe the indefinite of Timing: enum Timing { /** specifies that a duration, end or start time is indefinite*/ INDEFINITE, /** specifies a simple duration as the intrinsic media duration. This is only valid for elements that define media. */ MEDIA }; only and if only the eTiming is Timing_INDEFINITE, the animation has a fTiming, so the judgment "if ( eTiming == Timing_INDEFINITE )" may be unnecessary. @Steve Yin what's your opinion^_^? I'll take this one This defect duplicated with #i119711# *** This issue has been marked as a duplicate of issue 119711 *** |