[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Gnash-commit] gnash ChangeLog gui/NullGui.cpp
From: |
strk |
Subject: |
Re: [Gnash-commit] gnash ChangeLog gui/NullGui.cpp |
Date: |
Sat, 7 Jul 2007 16:33:09 +0200 |
On Sat, Jul 07, 2007 at 03:33:35PM +0200, Udo Giacomozzi wrote:
> Honestly, I think you're actually making a problem out of it. The name
> of the file/function is badly choosen. Ok, let's change that. However,
> the code itself is fine and no reason to go for big libraries like
> Boost. After all, we're just talking about simple timing functions...
The problem with tu_ was initially knowledge. I tought it was best to
use existing *standard* or de-facto standard classes rather then custom
ones, in particular to help a team of developers with a good documentation
and more peer-reviewd implementations of common algorithms.
I haven't seen the timer one, but that's a single function and I don't
have any big problem with it. If boost provides an equivalent function
"for free" (ie: not increasing deps) that's fine too for me.
Again: it's not the names, it's the lack of classes documentation, proper
copy constructors or assignment operators and so on... it's usually not-clean
code in general. And yes, I confirm we never discussed timers in particular
till now.
> usleep() is *not* a timing function. It's primary purpose is to tell
> the system scheduler that the process is not going to do anything
> within the next X nanoseconds. The scheduler then gives the next
> process the running state.
Could it be worth to use an inlined task_switch() call for clarity ?
> case) and it does not use tu_timer. I'd like to change the latter in
> favour to some generic timing but one has to tell me where I can find
> that. (I'd opt for tu_timer renamed to whatever name you prefer).
I think Markus is doing some research about ligher boost headers
for that.
--strk;
- Re: [Gnash-commit] gnash ChangeLog gui/NullGui.cpp, (continued)
- Message not available
- Re: [Gnash-commit] gnash ChangeLog gui/NullGui.cpp, Martin Guy, 2007/07/06
- Re: [Gnash-commit] gnash ChangeLog gui/NullGui.cpp, Martin Guy, 2007/07/06
- Re: [Gnash-commit] gnash ChangeLog gui/NullGui.cpp, Martin Guy, 2007/07/06
- Re: [Gnash-commit] gnash ChangeLog gui/NullGui.cpp, Tomas Groth, 2007/07/06
- Re: [Gnash-commit] gnash ChangeLog gui/NullGui.cpp, Martin Guy, 2007/07/06
- Re: [Gnash-commit] gnash ChangeLog gui/NullGui.cpp, zou lunkai, 2007/07/08
- Re: [Gnash-commit] gnash ChangeLog gui/NullGui.cpp, Sandro Santilli, 2007/07/08
- Re[2]: [Gnash-commit] gnash ChangeLog gui/NullGui.cpp, Udo Giacomozzi, 2007/07/06
- Re: Re[2]: [Gnash-commit] gnash ChangeLog gui/NullGui.cpp, Martin Guy, 2007/07/07
- Re[4]: [Gnash-commit] gnash ChangeLog gui/NullGui.cpp, Udo Giacomozzi, 2007/07/07
- Re: [Gnash-commit] gnash ChangeLog gui/NullGui.cpp,
strk <=
- Re: Re[4]: [Gnash-commit] gnash ChangeLog gui/NullGui.cpp, Martin Guy, 2007/07/07
- Re[6]: [Gnash-commit] gnash ChangeLog gui/NullGui.cpp, Udo Giacomozzi, 2007/07/07
- Re: Re[6]: [Gnash-commit] gnash ChangeLog gui/NullGui.cpp, Martin Guy, 2007/07/08
- Re: Re[6]: [Gnash-commit] gnash ChangeLog gui/NullGui.cpp, strk, 2007/07/08
- Re[8]: [Gnash-commit] gnash ChangeLog gui/NullGui.cpp, Udo Giacomozzi, 2007/07/09
- Re: [Gnash-commit] gnash ChangeLog gui/NullGui.cpp, Rob Savoye, 2007/07/10
- Re: [Gnash-commit] gnash ChangeLog gui/NullGui.cpp, Eric Hughes, 2007/07/10
[Gnash-commit] gnash ChangeLog gui/NullGui.cpp, Markus Gothe, 2007/07/09