|
Rockbox mail archiveSubject: RE: kevin: apps alarm_menu.cRE: kevin: apps alarm_menu.c
From: Anton Oleynikov <len0x_at_mail.ru>
Date: Wed, 16 Nov 2005 17:33:35 -0000 > What exactly makes this function called "sync"? My guess would be it syncs the message on multiple screens, because there is no parameter that specifies which screen message should be displayed on. > Also, why oh why didn't you just make your new function use > the same name? I really don't like seeing every function we have get a gui_ > prefix for no useful purpose. IMHO it's a good practice to give functions new names if they are totally rewritten because: - you don't break old code - you can migrate portions of code from old one to the new one and not everything in one go Anton. Received on 2005-11-16 Page template was last modified "Tue Sep 7 00:00:02 2021" The Rockbox Crew -- Privacy Policy |