[tarantool-patches] Re: [PATCH v2] fio: introduce utime function
Vladimir Davydov
vdavydov.dev at gmail.com
Tue Jul 9 11:55:08 MSK 2019
On Mon, Jul 08, 2019 at 06:52:24PM +0300, Олег wrote:
>
> Thanks! My bad. I really missed it.
>
> Please apply following diff to my patch.
>
> diff --git a/src/lua/fio.c b/src/lua/fio.c
> index 55fa66762..48e2f8eb3 100644
> --- a/src/lua/fio.c
> +++ b/src/lua/fio.c
> @@ -709,7 +709,7 @@ lbox_fio_utime(struct lua_State *L)
> const char *pathname = lua_tostring(L, 1);
> double atime = lua_tonumber(L, 2);
> double mtime = lua_tonumber(L, 3);
> - return lbox_fio_pushbool(L, coio_utime(pathname, atime, mtime) == 0);
> + return lbox_fio_pushbool(L, coio_utime(pathname, mtime, atime) == 0);
> }
So, now we have:
eio_utime(path, atime, mtime)
coio_utime(path, mtime, atime)
fio.utime(path, atime, mtime)
I think that for the sake of consistency coio_utime should take atime as
the second argument and mtime as the third.
Please fix and send v3 in a separate email.
More information about the Tarantool-patches
mailing list