Rockbox mail archiveSubject: RE: Latest Bookmarking Patch
From: Daniel Stenberg (daniel_at_haxx.se)
On Sat, 1 Feb 2003, Benjamin wrote:
Thanks for all your work on this feature. I do however have some remarks on
* It still contains CRLF newlines all over.
* On several places in the code you have changed a section like:
with your new code that looks like this:
Evidently, you alter the flow and I'm not convinced that you should do
* I suggest you use button_get(true) instead of button_get_w_tmo() if you're
* The bookmark_debug_print() function should probably be replaced with
Otherwise I think it looks mighty fine!
-- Daniel Stenberg -- http://rockbox.haxx.se/ -- http://daniel.haxx.se/
Page was last modified "Jan 10 2012" The Rockbox Crew