[m-rev.] for review: document leap seconds and target language

Julien Fischer jfischer at opturion.com
Sat Aug 14 03:43:41 AEST 2021


On Sat, 14 Aug 2021, Zoltan Somogyi wrote:

> 2021-08-14 00:13 GMT+10:00 "Julien Fischer" <jfischer at opturion.com>:
>> +    % For target languages that do not support leap seconds:
>> +    % - predicates in this module producing a `tm' value as an output will
>> +    % never set its `tm_sec' component to a value beyond 59.
>> +    % - predicates in this module taking a `tm_sec' value as an input will
>> +    % throw a time_error/1 exception if the value of the `tm_sec' component
>> +    % is beyond 59.
>
> The first item should end with "; and" instead of a period, because the period
> ends the applicability of the "For target languages ..." condition.

Done.

> Otherwise, the diff is fine.

Thanks for the review.

Julien.


More information about the reviews mailing list