On Friday 20 July 2012 17:11:33 Marek Vasut wrote: > Dear Mike Frysinger, > > On Friday 20 July 2012 07:33:49 Marek Vasut wrote: > > > Dear Mike Frysinger, > > > > On Wednesday 04 July 2012 10:39:20 Marek Vasut wrote: > > > > > > Putting there the __func__ name would improve structure and speed > > > > > > up finding right place. > > > > > > > > > > And if you want to use even __LINE__, look up __stringify patch in > > > > > the ML archives ;-) > > > > > > > > ugh, no, let's not use __LINE__ anywhere other than debug(). it has > > > > no business in code we ship as it is pointless bloated noise. > > > > > > Helps find out the problematic place in code ... > > > > except the code changes thus invalidating the line numbers, and how often > > are you putting the same string in multiple places that you can't easily > > coordinate where it came from ? if people are using the same exact > > string in multiple places, that sounds like a different problem. > > You can always replace the function names with macros, which expand in > place. And then simply add __func__ __LINE__ __FILE__ etc.
if you wanted to add it while debugging, that's fine, but the point is that this doesn't belong in normal runtime images. it's even trivial to define such a macro: #ifdef DEBUG # define printf(fmt, args...) printf("%s:%s:%i: " fmt, __FILE__, __LINE__, __func__, ## args) #endif (will obviously need a little more work to handle non-const fmt strings, but you get the idea). -mike
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot