Bug 69467 - outline view is toast in 4.2
Summary: outline view is toast in 4.2
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Impress (show other bugs)
Version:
(earliest affected)
4.2.0.0.alpha0+ Master
Hardware: Other All
: medium normal
Assignee: Caolán McNamara
URL:
Whiteboard: target:4.2.0
Keywords:
Depends on:
Blocks: mab4.2
  Show dependency treegraph
 
Reported: 2013-09-17 13:32 UTC by Caolán McNamara
Modified: 2013-10-11 11:51 UTC (History)
3 users (show)

See Also:
Crash report or crash signature:


Attachments
busted outline, text overlaps image (69.98 KB, image/png)
2013-09-17 13:32 UTC, Caolán McNamara
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Caolán McNamara 2013-09-17 13:32:06 UTC
Created attachment 85982 [details]
busted outline, text overlaps image

broken in e2aa5ab249edaeb8e9e3603ba8ae09faea90122c, unknown first broken commit
Comment 1 Caolán McNamara 2013-10-09 18:13:07 UTC
triggered by 43b84bbba2519686eb6b0463f422bf695d2d74b9 I believe
Comment 2 Caolán McNamara 2013-10-10 08:19:46 UTC
The problem is triggered by the change from 3990 to 
#define OWN_ATTR_VALUE_END 3991 of include/svl/solar.hrc in 43b84bbba2519686eb6b0463f422bf695d2d74b9 there must be overlapping ids in there somewhere :-(
Comment 3 Caolán McNamara 2013-10-10 20:07:48 UTC
oh god, what is this vomit in EditEngineItemPool::EditEngineItemPool e.g. aV5Map
Comment 4 Miklos Vajna 2013-10-11 07:48:05 UTC
Strange, last time OWN_ATTR_VALUE_END was modified in 5924454a867bd500fdc696bb9939c84425c5e025, no editeng change was necessary, as far as I see.

Regarding the editeng hardwired which ids, I believe those are (were?) used for some kind of binary import/export, 305c3ee0d0b4c49de8e8829a99da9a17063363a9 is an example on how a new map can be added when the ids change. But I suspect the problem is somewhere else, if no mapfile was added for 5924454a867bd500fdc696bb9939c84425c5e025, either.
Comment 5 Miklos Vajna 2013-10-11 08:16:19 UTC
One more interesting thing is that if OWN_ATTR_VALUE_END is changed back to 3990, then everything is fine (no failing unit tests, even if code added by 43b84bbba2519686eb6b0463f422bf695d2d74b9 is covered by unit tests; also outline view is OK). But I guess we want to know exactly what relies on an unchanged OWN_ATTR_VALUE_END and fix *that* instead.
Comment 6 Commit Notification 2013-10-11 10:04:49 UTC
Caolan McNamara committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=0a0248f89048fb3d85696eddbbcb2c176bc468c5

Resolves: fdo#69467 nTxtLeft not compared in operator==



The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds
Affected users are encouraged to test the fix and report feedback.
Comment 7 Caolán McNamara 2013-10-11 11:51:37 UTC
v. fishy