[Lazarus] CHM help package

Sergey Bodrov serbod at gmail.com
Tue Dec 12 14:46:40 CET 2017


2017-12-12 16:32 GMT+03:00 Werner Pamler via Lazarus <
lazarus at lists.lazarus-ide.org>:

> Am 12.12.2017 um 11:01 schrieb Sergey Bodrov via Lazarus:
>
> BTW,  in ipcss.inc TCSSProps.ReadCommands() lack of Args.Count checking
> for 'border-width', 'border-color' and 'border-style' cause Args[0]
> out-of-range errors.
> Instead of tons checks for Args.Count, I propose single local function,
> that returns string from Args by index, or empty string if there is no such
> index.
>
>
> Really? It looks fine for me. I even removed some more checks for
> Args.Count > 0 when accessing Args[0] - this code is never reached because
> it happens inside an Args loop which is never entered if Args.Count = 0.
> What did you do to see that there is a problem?
>

Maybe, you confused with ACommands loop? There is no loops for Args.


-- 
*Bodrov Sergey*
software development, IT consulting
http://www.serbod.com
*Phone (Belarus):* +375(25)794-21-58
*Skype:* sergey.bodrov1
*e-mail:* serbod at gmail.com, oxotnuk at yandex.ru
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.lazarus-ide.org/pipermail/lazarus/attachments/20171212/586e1f7d/attachment.html>


More information about the Lazarus mailing list