|
From: Daniel J S. <dan...@ie...> - 2006-03-10 08:13:43
|
I have a bit of free time to wrap up the patch to stop the constant palette refresh with X11 mouse activity. Just one detail and I wanted to get developers opinions. As part of a palette comparison a copy of the previous palette is stored. Part of that is copying a udft_entry. Right now it is only a partial copy of udft_entry as the color routines don't use all parts of the udft_entry in the test. I'm just wondering which of these alternatives are preferred: 1) Leave this code inside getcolor.c as an incomplete copy and call the routines: udft_partial_cpy() udft_partial_del() 2) Make the routines a complete copy and delete keeping track of details for temp_at, perm_at, free_at and move them to somewhere like eval.h/eval.c so that they might find some future use somewhere else. Basically, do you want a udft_cpy() udft_del() ? Dan |