Bug ID: 392952
           Summary: Regression: port to QTimeZone leads to blowing up iCal
                    file size
           Product: kcalcore
           Version: git
          Platform: Other
                OS: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: NOR
         Component: general
  Target Milestone: ---

Created attachment 111933
fix for repeated VTIMEZONE entries in iCal resource

introduced an unintended side effect that is blowing up iCal file sizes by
storing a VTIMEZONE entry per event instead of a consolidated list.

The issue is ultimately caused by the replacement of the QMap based
ICalTimeZones class with a QVector based type for tzUsedList. The
ICalTimeZones::add method did check whether a to be added timezone is already
in the map, whereas the replacement QVector::push_back method does not.

The bug occurs in lines 2419-2421 (Application/17.12 branch) of
icalformat_p.cpp and is easily fixed with the attached patch.

You are receiving this mail because:
You are the assignee for the bug.

Reply via email to