Bug 48140 - Remove uno calls from xlsx import
Summary: Remove uno calls from xlsx import
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Spreadsheet (show other bugs)
Version: Master old -3.6
Hardware: Other All
: medium normal
Assignee: Not Assigned
QA Contact:
URL:
Whiteboard: EasyHack DifficultyBeginner SkillCpp
Keywords:
Depends on:
Blocks:
 
Reported: 2012-03-31 19:17 UTC by Markus Mohrhard
Modified: 2015-01-03 17:38 UTC (History)
4 users (show)

See Also:
i915 platform:
i915 features:


Attachments

Description Markus Mohrhard 2012-03-31 19:17:31 UTC
Noel moved the xlsx import code from oox to sc which allows us to use calc core methods directly and therefore get much better performance.

We should start by moving some of the more important parts to direct calls.

Good starting points are range name import, database range import, cell value/string import, ...

The source code for xlsx import is now in http://opengrok.libreoffice.org/xref/core/sc/source/filter/oox/

It is a good idea to check how the corresponding ods import is done at http://opengrok.libreoffice.org/xref/core/sc/source/filter/xml/
Comment 1 Daniel Bankston 2012-04-05 07:45:41 UTC
Hi, Markus,

Per my IRC conversation with you and Kohei, I'll be looking into this.
Comment 2 Daniel Bankston 2012-04-14 21:22:36 UTC
I have submitted a patch to the mailing list that partially addresses this EasyHack.  More patches will be forthcoming.
Comment 3 Stéphane Aulery 2012-08-11 01:25:15 UTC
Deteted "Easyhack" from summary
Comment 4 Elton Chung 2013-06-26 09:59:54 UTC
Just a note (previous commit regarding this bug):
Converting UNO calls to direct in WorkbookHelper

http://cgit.freedesktop.org/libreoffice/core/commit/?id=f892be678a036cd0fea665eda529e9cc906e0cfa
Comment 5 Björn Michaelsen 2013-10-04 18:46:22 UTC
adding LibreOffice developer list as CC to unresolved EasyHacks for better visibility.

see e.g. http://nabble.documentfoundation.org/minutes-of-ESC-call-td4076214.html for details
Comment 6 bfoman 2014-02-20 18:25:26 UTC
This bug is confirmed but unowned - back to NEW.
Comment 7 Alex Thurgood 2015-01-03 17:38:08 UTC
Adding self to CC if not already on


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.