Date: prev next · Thread: first prev next last
2014 Archives by date, by thread · List index


Hi Stephan, Hi List,

sorry, I guess I misspoke:
Those examples, which I listed, are rather obvious problems that aren't translation related.
So let me rephrase:
        How to call a developer's attention to small pieces of code?
        And especially: may I combine that with translation patches?


I thought "OSL_ENSURE(false, …)" would be "problematic", because firstly there is an "OSL_FAIL" 
definition in "include/osl/diagnose.h", which seems to do exactly that.
Secondly, the place I saw this (cui/source/tabpages/numpages.cxx line 2791 ff.) encapsulates that 
statement in an if clause.
Together with the error message ("cannot happen"), it implies that could be reduced from…
--- snip ---
    if (SVX_MAX_NUM <= nLvl)
    {
        OSL_ENSURE(false, "cannot happen.");
        return;
    } 
--- snap ---
… to something like this:
--- snip ---
        OSL_ENSURE_RETURN(!(SVX_MAX_NUM <= nLvl), "cannot happen.");
--- snap ---
(with OSL_ENSURE_RETURN being a OSL_ENSURE including a return statement.)

If OSL_ENSURE is deprecated anyway, I guess this is irrelevant anyway.


Yours,
Sven

On Friday 17 January 2014 12:08:15 Stephan Bergmann wrote:
[would probably have been better to start a new mail thread]

On 01/17/2014 11:34 AM, Sven Wehner wrote:
When I scroll through the source code, I sometimes see odd code.
Things that a real developer should seriously take a look at.
Something like one of the multiple "DELETEZ" defines, an "OSL_ENSURE(false, \"cannot 
happen.\")", German function names etc.

Not sure what "a real developer should seriously take a look at" in an

   OSL_ENSURE(false, "cannot happen.")

(other than OSL_ENSURE being deprecated).  Those are generally 
assertions that program logic cannot reach that point (like a default 
case in a switch statement) unless there is a bug.

Stephan



Context


Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.