Hallo Chris,

it is not the amount of code, it is the maintenance of such.

If you use the code in 3 differnet files and you change only one line, 
you will have to edit three files to get the same behavior.
IMHO not using code twice is the cleaner style.

Norbert

Am 30.12.2016 um 16:09 schrieb Chris Morley:
> It'S own module  seems alot of work for little gain. It's about 10 lines of 
> code or so. It can be copy and pasted into whatever program needs it. 
> Changing the camel case name to not would suit the surrounding code ( 
> nitpicking )
> Chris M
>
> ----- Reply message -----
> From: "Jim Craig" <jimcraig5...@windstream.net>
> To: "EMC developers" <emc-developers@lists.sourceforge.net>
> Subject: [Emc-developers] tooledit_widget modifications
> Date: Fri, Dec 30, 2016 6:22 AM
>
>
>
> On 12/29/2016 9:38 AM, Niemand Sonst wrote:
>> Hallo Jim,
>>
>> first I would say thanks for your work, I am compiling at the moment to
>> test from my side.
>> I checked the code and I would like to recommend:
>>
>> Is it possible to get the code of:
>>
>> onTreeNavigateKeyPress
>>
>> in a own module? This way we would be able to use the same code also for
>> the offsetwidget and may be future widgets using a treeview.
>>
> Norbert,
>
> I am not sure that it is possible to put this in it's own module. It
> depends on a specific function for the tooledit widget to accept the
> values in the table to a specific format. I will finish making this
> function the way everyone likes then I will look at pulling it out into
> it's own module.
>
> Thanks,
>
> Jim
>
> ------------------------------------------------------------------------------
> Check out the vibrant tech community on one of the world's most
> engaging tech sites, SlashDot.org! http://sdm.link/slashdot
> _______________________________________________
> Emc-developers mailing list
> Emc-developers@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/emc-developers
>
> ------------------------------------------------------------------------------
> Check out the vibrant tech community on one of the world's most
> engaging tech sites, SlashDot.org! http://sdm.link/slashdot
> _______________________________________________
> Emc-developers mailing list
> Emc-developers@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/emc-developers
>


------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most 
engaging tech sites, SlashDot.org! http://sdm.link/slashdot
_______________________________________________
Emc-developers mailing list
Emc-developers@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/emc-developers

Reply via email to