Hey Goetz,
2008/7/14 Goetz Babin-Ebell <[EMAIL PROTECTED]>:
> Hello Folks,
>
> Index: src/wt/WLogger.C
> ===================================================================
> RCS file: /opt/cvs/wt/src/wt/WLogger.C,v
> retrieving revision 1.1
> diff -u -r1.1 WLogger.C
> --- src/wt/WLogger.C 13 Jul 2008 18:15:10 -0000 1.1
> +++ src/wt/WLogger.C 14 Jul 2008 17:19:13 -0000
> @@ -144,6 +144,7 @@
>
> bool WLogEntry::Impl::quote() const
> {
> + if (logger_.fields().empty()) return false;
> return logger_.fields()[currentField_].isString();
> }
Although it was a bit assuming, it shouldn't happen unless you are
trying to create too many fields considering the number of
WLogger::addField() calls you have done?
If not, there is a bug ?
If so, then, yes, perhaps it is okay to relax the constraint and allow
to add fields to a logger which haven't been explicitly declared --
but it is perhaps not a good idea?
Regards,
koen
-------------------------------------------------------------------------
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
_______________________________________________
witty-interest mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/witty-interest
