Hi Thorsten,

>> Of course, some kind of resource-fed versions of those would also be
>> nice :), with a similar reason as above: Exception messages are
>> potentially read by end users, and sometimes it's debatable whether they
>> should be localized.
>>
> Nope, don't think so. Lingu franca of programming is English - or do
> you also want to translate IDL documentation?

I did say this is a can of worms, didn't I? :) Let's not discuss it in
this thread here ...

>>>           OSL_ENSURE(false, m); \
>>>           return false; }
>> parametrizing the return type might be a good idea, too.
>>
> One has to strike a balance. I thought about it as well, but the cases
> I would have needed a different return type (mostly NULL) have been
> seldom enough that I wouldn't impose the additional clutter to the
> macro... 

Okay.

>> Well, boost/current_function.hpp is really small. Nobody hinders us from
>> duplicating those about 10 lines (less, if we count currently supported
>> compilers only) of code in UDK, right?
>>
> Nope. Would prolly also benefit the other assertions - I'd change them
> to include the function name then, as well.

+1

>> (Yes, they *belong* into the UDK. We should definitely *not* introduce
>> yet another place of such diagnostics facilities. We already have
>> osl/diagnose.h, tools/debug.hxx (and, speaking strictly, the less known
>> tools/diagnose_ex.h), and those are already too much.)
>>
> So, you suggest using stuff from udkapi in osl, relying on the fact
> that anyone making sensible use of them prolly implements UNO API (and
> thusly already is past udkapi in the build hierarchy)? 

Ehm, well, I overlooked your argument "osl is too deep for UNO" in the
first place. Okay, this perhaps wouldn't be a good idea (and vetoed by
Stephan, anyway :). Sigh. I really really really really dislike the fact
of introducing yet another place for diagnostics. So much that I'm
tempted to say we should not do this at all :-\

Okay, what's in the UDK, and able to use UNO? Let's place a diagnose.h
there, which also includes osl/diagnose.h.

Ciao
Frank

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to