Bug 35406

Summary: simplify lcl_GetRelativePos
Product: LibreOffice Reporter: Caolán McNamara <caolanm>
Component: SpreadsheetAssignee: Kohei Yoshida (inactive) <libreoffice>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: medium    
Version: unspecified   
Hardware: Other   
OS: All   
Whiteboard:
i915 platform: i915 features:
Attachments: like so

Description Caolán McNamara 2011-03-18 01:53:42 UTC
Created attachment 44568 [details]
like so

In sc/source/ui/unoobj/shapeuno.cxx we have a
ScRange aRange = ... where aRange is then unused, it seems the entire line doesn't do anything, which leaves a symmetrical expression that can be simplified a bit, as attached.
Comment 1 Kohei Yoshida (inactive) 2011-03-18 07:51:01 UTC
Yup, the patch makes sense, and it's a lot simpler.

I've committed it locally, and as soon as kendy is done merging the integration/dev300_m101 branch onto master I'll push.
Comment 2 Kohei Yoshida (inactive) 2011-03-18 08:41:19 UTC
Merging is done.  Pushed to master.

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.