D15428: [textlayout] Don't enter infinite loop when table is misfit

2018-09-13 Thread Anthony Fieroni
anthonyfieroni updated this revision to Diff 41564. anthonyfieroni added a comment. Another try: Don't mark table as misfit when it does not have header rows REPOSITORY R8 Calligra CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D15428?vs=41418&id=41564 REVISION DETAIL https://

D15412: [filters] Extend table lifetime

2018-09-13 Thread Jarosław Staniek
staniek accepted this revision. staniek added a comment. This revision is now accepted and ready to land. Good job! REPOSITORY R8 Calligra REVISION DETAIL https://phabricator.kde.org/D15412 To: anthonyfieroni, danders, boemann, #calligra:_3.0, staniek Cc: staniek, Calligra-Devel-list, dc

D15412: [filters] Extend table lifetime

2018-09-13 Thread Anthony Fieroni
anthonyfieroni updated this revision to Diff 41552. REPOSITORY R8 Calligra CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D15412?vs=41377&id=41552 REVISION DETAIL https://phabricator.kde.org/D15412 AFFECTED FILES filters/libmsooxml/MsooXmlDrawingReaderTableImpl.h filters/libmso

D15412: [filters] Extend table lifetime

2018-09-13 Thread Jarosław Staniek
staniek added inline comments. INLINE COMMENTS > anthonyfieroni wrote in MsooXmlDrawingReaderTableMethods.h:40 > That isn't a problem, it's initialized at constructor time. I know but original code was not called new KoTable for PptxXmlSlideReader, right? > anthonyfieroni wrote in DocxXmlDocum

D15412: [filters] Extend table lifetime

2018-09-13 Thread Anthony Fieroni
anthonyfieroni added a comment. It can be allocated only in read_tbl but as you can see in bug report, use-after-free is very common to happen. INLINE COMMENTS > staniek wrote in MsooXmlDrawingReaderTableMethods.h:40 > This is declaration-only header, so better no, otherwise allocation of a

D15412: [filters] Extend table lifetime

2018-09-13 Thread Jarosław Staniek
staniek requested changes to this revision. staniek added a comment. This revision now requires changes to proceed. Thanks, lots of other people also touched this code but I tried this time to review :) I propose to separate the string literal changes. Also I wonder why there's "Context n

D15412: [filters] Extend table lifetime

2018-09-13 Thread Camilla Boemann
boemann added a comment. No idea either - jaroslaw was the original author so maybe he can take a look REPOSITORY R8 Calligra REVISION DETAIL https://phabricator.kde.org/D15412 To: anthonyfieroni, danders, boemann, #calligra:_3.0 Cc: Calligra-Devel-list, dcaliste, cochise, vandenoever

D15412: [filters] Extend table lifetime

2018-09-13 Thread Dag Andersen
danders added a comment. Sorry, don't have any qualified opinons, I don't quite understand the original code either, so... Imo if you think it is good, land it. REPOSITORY R8 Calligra REVISION DETAIL https://phabricator.kde.org/D15412 To: anthonyfieroni, danders, boemann, #calligra:_3