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

[Pavel Emelyanov]

On 02/08/2012 07:06 PM, Kinsbursky Stanislav wrote:
> 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().

cr_fdset_open(what_for)