[Pki-devel] [PATCH] 0015-Removal-of-unused-private-methods
Adam Young
ayoung at redhat.com
Fri Dec 9 02:25:52 UTC 2011
On 12/08/2011 03:00 PM, Adam Young wrote:
> On 12/07/2011 11:07 AM, Adam Young wrote:
>> On 11/16/2011 10:38 AM, Adam Young wrote:
>>> On 11/16/2011 09:48 AM, Ade Lee wrote:
>>>> I'm struggling with the premise behind this patch. Some of these
>>>> methods seem like they are valuable - and certainly they were valuable
>>>> at some point in time (or they would not have been written). On the
>>>> other hand, I see the benefit of trimming the code base. No need to
>>>> refactor code that isn't being used.
>>>>
>>>> I realize that this removes some eclipse warnings, but is removing
>>>> these
>>>> methods good practice?
>>>>
>>>> I guess I'm looking for some other folks to chime in here.
>>>
>>> My feeling is that you do not leave dead code in the code base. The
>>> code still exists in the repository, but it is unlikely that a
>>> piece of code that is not called today will be needed in exactly the
>>> same form in the future.
>>>
>>>
>>> Code is documentation. Code that is not used is misleading to the
>>> maintainer. Committing commented out code is a bad practice.
>>>
>>> As we refactor, the location of the behavior of a lot of these
>>> methods will change. Thus, some orphan calls will become
>>> uncallable, and others will get in the way of code clean up.
>>>
>>>
>>> This is old code: some of it goes back 15 years. Holding on to
>>> unused, uncallable code is akin to hoarding.
>>>
>>>
>>>>
>>>> Ade
>>>>
>>>> On Thu, 2011-11-10 at 22:29 -0500, Adam Young wrote:
>>>>> _______________________________________________
>>>>> Pki-devel mailing list
>>>>> Pki-devel at redhat.com
>>>>> https://www.redhat.com/mailman/listinfo/pki-devel
>>>>
>>>
>>> _______________________________________________
>>> Pki-devel mailing list
>>> Pki-devel at redhat.com
>>> https://www.redhat.com/mailman/listinfo/pki-devel
>>
>>
>> Conflicted with the PKI Silent changes, so patch has been remade by
>> hand.
>>
>>
>>
>> _______________________________________________
>> Pki-devel mailing list
>> Pki-devel at redhat.com
>> https://www.redhat.com/mailman/listinfo/pki-devel
> Withdrawn., The formatting changes make it impractical to try and
> reformat this. Will be resubmitted in smaller patches shortly.
>
>
> _______________________________________________
> Pki-devel mailing list
> Pki-devel at redhat.com
> https://www.redhat.com/mailman/listinfo/pki-devel
Rebased on top of the format changes
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/pki-devel/attachments/20111208/f1f62d9b/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: dogtag-admiyo-0015-2-Removal-of-unused-private-methods.patch
Type: text/x-patch
Size: 129468 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/pki-devel/attachments/20111208/f1f62d9b/attachment.bin>
More information about the Pki-devel
mailing list