Bug 36230 - [EasyHack] Remove dead code aGotoCallButton and RID_IMGBTN_GOTOCALL
Summary: [EasyHack] Remove dead code aGotoCallButton and RID_IMGBTN_GOTOCALL
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: BASIC (show other bugs)
Version: unspecified
Hardware: Other All
: medium normal
Assignee: Noel Power
QA Contact:
URL:
Whiteboard: EasyHack DifficultyBeginner SkillCpp
Keywords:
Depends on:
Blocks:
 
Reported: 2011-04-14 03:58 UTC by Christian Dywan
Modified: 2011-07-28 10:00 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
Remove unused aGotoCallButton and RID_IMGBTN_GOTOCALL (3.32 KB, patch)
2011-04-14 04:18 UTC, Christian Dywan
Details | Splinter Review

Description Christian Dywan 2011-04-14 03:58:25 UTC
The AGotoCallButton is never shown and basically unused code, and accordingly RID_IMGBTN_GOTOCALL. So it should be removed.

See http://wiki.documentfoundation.org/Development/Easy_Hacks#Remove_obsolete_product_pieces
Comment 1 Christian Dywan 2011-04-14 04:18:24 UTC
Created attachment 45602 [details] [review]
Remove unused aGotoCallButton and RID_IMGBTN_GOTOCALL
Comment 2 Noel Power 2011-04-15 02:52:43 UTC
ok, taking it
Comment 3 Noel Power 2011-04-18 05:17:21 UTC
Thanks for the patch, it's pushed ( I also removed the id completely from the source (  svx/inc/svx/svxids.hrc )

commits are here

http://cgit.freedesktop.org/libreoffice/components/commit/?h=libreoffice-3-4&id=7eba32f0eacd71fbf2561c9f58f52b89cdd0e86f
http://cgit.freedesktop.org/libreoffice/libs-core/commit/?h=libreoffice-3-4&id=31dd8b55accd586d454280814b1ad856802454ef
Comment 4 Noel Power 2011-04-18 05:17:37 UTC
mark as fixed
Comment 5 Björn Michaelsen 2011-07-28 10:00:21 UTC
=== Remove specific dead code ===

'''Background:''' We have lots of dead code around, one example is: basicide/source/basicide/baside2b.cxx - where we have code related to RID_IMGBTN_GOTOCALL that creates an aGotoCallButton that has no icon and is always hidden. We should remove both the code, the .src and .hrc defines.

'''Skills:''' simple C++

'''Finished the part of removing aGotoCallButton''': Christian Dywan, christian.dywan@lanedo.com, 2011-04-21 [https://bugs.freedesktop.org/show_bug.cgi?id=36230 fdo#36230]


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.