[ros-dev] Re:
[ros-svn] [gdalsnes]18113:-reorderInsertXscendingOrder
macro argument order andupdate uses
Thomas Weidenmueller
w3seek at reactos.com
Wed Sep 28 18:14:55 CEST 2005
Casper Hornstrup wrote:
> Oh please don’t go there. Do you honestly think you (in general) are
> smarter at generating microprocessor instructions than a multi-million hour
> compiler software?
In some places it does matter, especially in the kernel.
Splitting up a 200 lines functions in 10 helper functions IMO is ugly,
especially when those helper functions are only used by that one single
function. To avoid code duplication I *do* create helper functions, but
most often only for things that can be shared with other code. I don't
have problems with functions with a few hundreds of lines of code as
long as the coding style is acceptable and it is sufficiently documented
with comments.
However, we're going off topic...
- Thomas
More information about the Ros-dev
mailing list