This patch was a great idea! "Timer ticks" has been annoying me for quite
some time now. (No pun intended.)
However, you shouldn't be putting so much code in that handler. You should
be writing a seperate function that uses StringBuilder to build a CString8
to display that contains the formatted time data. And then you should be
printing that in the tick handler or whatever.
To encourage better coding practices, I'm going to ask that you do this and
submit the improved version for me to commit.
However, keep in mind - code you add to the kernel is generally yours to
maintain, if at all possible. So that means when xfury finishes coding his
software clock stuff, you need to consider whether or not you (as in we)
want your code to display software time or hardware time, on every single
tick.
Regardless, timer ticks themselves are on their way out, and this is a great
improvement, once its done properly.
-------------------------------------------------------------------------
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
_______________________________________________
SharpOS-Developers mailing list
SharpOS-Developers@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/sharpos-developers
