On Thu, Jan 04, 2007 at 09:26:45AM +1100, David Gibson wrote:
> On Wed, Jan 03, 2007 at 02:20:38PM -0800, Nishanth Aravamudan wrote:
> > On 03.01.2007 [10:27:19 +1100], David Gibson wrote:
> > > On Tue, Jan 02, 2007 at 03:03:00PM -0800, Nishanth Aravamudan wrote:
> > > > Some of our error and debugging paths are missing newlines, which makes
> > > > for confusing output when things are misconfigured. Make sure all
> > > > ERROR and WARNING in the library is properly terminated.
> > >
> > > Actually, I think the right way to fix this is to remove \n from any
> > > ERROR() and WARNING() macros that do have then, and add them back in
> > > the macro itself. I'm pretty sure the reason I've made these errors
> > > is because I was assuming that ERROR() and WARNING() added the \n,
> > > like CONFIG() and FAIL(), which in turn work that way because perror()
> > > does.
> >
> > I disagree with this, just because it means we, for instance, in the
> > case with my other patch to *remove* a newline, will now need to provide
> > another set of non-newlined macros...
>
> Hrm, yes. Ok, I take that back.
Gah, sorry. My replies keep getting delayed, because my mail setup is
such that I get the list mails to my external address, but my replies
go through the ibm address. I'm on leave at the moment so the replies
get held until I remember to turn on the vpn.
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Libhugetlbfs-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/libhugetlbfs-devel