[Tarantool-patches] [PATCH luajit 2/2] Make check in os.time() consistent.

[Evgeniy Temirgaleev]

Hi, Sergey! Thanks for the review!
Fixed your comments and force-pushed the branch.

> 
> From: Sergey Kaplun <skaplun at tarantool.org>
> To: Evgeniy Temirgaleev <e.temirgaleev at tarantool.org>
> Cc: Sergey Bronnikov <sergeyb at tarantool.org>, tarantool-patches at dev.tarantool.org
> 
> Date: Tuesday, June 30, 2026 1:09 PM +03:00
> Hi, Evgeniy!
> Thanks for the patch!
> LGTM, with a few nits regarding the commit message.
> 
> On 29.06.26, Evgeniy Temirgaleev wrote:
> > From: Mike Pall <mike>
> >
> > Thanks to Temir Galeev. #1470
> 
> Please strip the issue number here. See the rationale in the response to
> the previous patch.
> 

Removed.

> 
> 
> >
> > (cherry picked from commit 295d45fb26de56498782c94594f31b97aef744ef)
> >
> > This patch prevents os.time() to return nil, when machine time is -1
> 
> Typo: s/to return nil/from returning nil/
> 

Fixed.

> 
> 
> > and some error occurs before os.time() call. In fact, this is
> 
> Typo: s/os.time() call/the os.time() call/
> 

Fixed.

> 
> 
> > a theoretical case, so no tests were added.
> >
> > Temir Galeev:
> > * added the description for the patch
> >
> > Part of tarantool/tarantool#12480
> > ---
> 
> <snipped>
> 
> --
> Best regards,
> Sergey Kaplun
> 

--
Best regards,
Evgeniy Temirgaleev
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.tarantool.org/pipermail/tarantool-patches/attachments/20260630/dd05dd8b/attachment.htm>