Issue 112502 - desktop: unused code in desktop
Summary: desktop: unused code in desktop
Status: CONFIRMED
Alias: None
Product: General
Classification: Code
Component: code (show other issues)
Version: DEV300m83
Hardware: All Linux, all
: P3 Trivial (vote)
Target Milestone: ---
Assignee: AOO issues mailing list
QA Contact:
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-06-18 09:58 UTC by caolanm
Modified: 2017-05-20 11:27 UTC (History)
1 user (show)

See Also:
Issue Type: PATCH
Latest Confirmation in: ---
Developer Difficulty: ---

Attachments
patch to remove unused code (35.89 KB, patch)
2010-06-18 09:59 UTC, caolanm 		no flags Details | Diff
expand to remove newly unused methods as of m84 (44.49 KB, patch)
2010-06-30 20:51 UTC, caolanm 		no flags Details | Diff
update for extra unused entries as of m87 (49.21 KB, patch)
2010-09-04 16:05 UTC, caolanm 		no flags Details | Diff
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description caolanm 2010-06-18 09:58:51 UTC 
remove final unused code from desktop

(as an aside, there are *two* xml_parse methods, one is used, but the other is
unused)

desktop/source/migration/cfgfilter.cxx
desktop/source/migration/cfgfilter.hxx
desktop/source/migration/services/autocorrmigration.cxx
desktop/source/migration/services/autocorrmigration.hxx

can be hg removed after applying. I guess its possible that
AutocorrectionMigration and CConfigFilter are supposed to be used by something,
but they aren't
Comment 1 caolanm 2010-06-18 09:59:20 UTC 
Created attachment 70071 [details]
patch to remove unused code
Comment 2 caolanm 2010-06-30 20:51:57 UTC 
Created attachment 70329 [details]
expand to remove newly unused methods as of m84
Comment 3 caolanm 2010-06-30 20:52:46 UTC 
as of m84
source/deployment/gui/dp_gui_system.cxx
source/deployment/gui/dp_gui_system.hxx
could also be removed
Comment 4 caolanm 2010-09-04 16:05:15 UTC 
Created attachment 71558 [details]
update for extra unused entries as of m87
Comment 5 Rob Weir 2013-03-11 15:03:33 UTC 
I'm adding this comment to all open issues with Issue Type == PATCH.  We have 220 such issues, many of them quite old.  I apologize for that.  

We need your help in prioritizing which patches should be integrated into our next release, Apache OpenOffice 4.0.

If you have submitted a patch and think it is applicable for AOO 4.0, please respond with a comment to let us know.

On the other hand, if the patch is no longer relevant, please let us know that as well.

If you have any general questions or want to discuss this further, please send a note to our dev mailing list:  dev@openoffice.apache.org

Thanks!

-Rob
Comment 6 Marcus 2017-05-20 11:27:27 UTC 
Reset assigne to the default "issues@openoffice.apache.org".