[CRIU] Re: [PATCH] crtools: cleanup fdset initalization

[Kinsbursky Stanislav]

08.02.2012 18:43, Pavel Emelyanov пишет:
> On 02/08/2012 06:29 PM, Kinsbursky Stanislav wrote:
>> This patch:
>> 1) Updates function cr_fdset_open() to be suitable for handling fdset creation
>> for dump and show stages.
>> 2) Replaces cr_fdset_open() by new wrapper function cr_fdset_dump().
>> 3) Replaces prep_cr_fdset_for_restore() by new wrapper function cr_fdset_show().
> Стас, вот ты сейчас опять, саркастически улыбаясь, отметишь, что это все вкусовщина,
> но, как может функция, которая *открывает* пачку файлом называться cr_fdset_*dump* ?!
>
> А?

Нет, в этом случае я отмечу иное: cr_fdset_open() тоже не очень подходит, 
учитывая, что в этой функции в файл ещё и пишут. Однако почему-то это тебя не 
парило особо.
Давай тогда поменяем на cr_get_dump_fdset() и cr_get_show_fdset().
Нормально?
Или даже что-нить типа cr_get_fdset_for_dump() и cr_get_fdset_for_show().

-- 
Best regards,
Stanislav Kinsbursky