From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.1 (2015-04-28) on sa.local.altlinux.org X-Spam-Level: X-Spam-Status: No, score=-2.0 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FROM autolearn=ham autolearn_force=no version=3.4.1 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20221208; t=1684095144; x=1686687144; h=content-transfer-encoding:in-reply-to:from:content-language :references:to:subject:user-agent:mime-version:date:message-id:from :to:cc:subject:date:message-id:reply-to; bh=gNgFabpNZVv3Sb/FkY8GJL4ZqVm/CFvAkh/Th3foRd0=; b=W8tafbdnJ+1U+tgMxcm88j3tAfJDooJeNCO/ZYl1NijAyZgkqZHJLR4ybSZWMoStYv bQRKAx13o+Q2SkWMrNovuAb8wXIre6BbbqoW5nP4SRd6GGBsF6Rv9P3A86BOggc7/8z7 jevTSnbeZgfS6FS4lQipuSaq3BGfgalMYE5Ks5UYI/9K4PPvH2JAbNo+Xs8jSSh7G5BN gmlCdr86xIVtsDB44F6Cm1LV5sDVWEZ8pXHxaOl1dpXtE+l1tbJRwt0AphCTT0w1G24H ElPbmW3NaWIXyF0tfWg4+Ehdk71s3Kazj21b2b/bVO6zVboH8r/7bhEDauGQENSlOwhZ VqPQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1684095144; x=1686687144; h=content-transfer-encoding:in-reply-to:from:content-language :references:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=gNgFabpNZVv3Sb/FkY8GJL4ZqVm/CFvAkh/Th3foRd0=; b=fyrZKFpTUOrFzT4oIOECFTb/7tfl2kIscqFW61bTxfzwXlGIUsVm9N+aQ1UkrxqW3Z +hm/xZMh1BxRDqRTR5Kg7evR4r8vtRZ8dIWX119vYEu2oGXEtPloNgrDIf4vWgj9a46m AXqIU9CKgBOWzSZuozGzvXF6XfHwXKToSm4BlWG4VBlL1ie6C6YsTJhePJznjSqJpyZi EWhoD63n7j3JF1/LyPewuu4A6agwzXK0ck+nLfx9oppwdiE1xA5/2gYz0jVHhR9I0vC7 ZSGprhc3tBmovsZ5zfiLjUdr3rYiV7QZ+yFNHaimfvjcnJwg332h2po2OA2Yrtsilcbt Ovdg== X-Gm-Message-State: AC+VfDzfMqeE0jObokyFc118HsUWl7m9FSGChlAl2l3uZGeXLk2MiDtG MtpyCzCsmVKLdIItbztyjwCAMdcVY4Q= X-Google-Smtp-Source: ACHHUZ7zCx241Ndsv9seL4xA1LzcZ57KK8KzoENRicq7heDZnhhNuMYH6veUfACtcO0EOr80xpIZrg== X-Received: by 2002:a25:2417:0:b0:ba7:490b:8b69 with SMTP id k23-20020a252417000000b00ba7490b8b69mr5010626ybk.32.1684095144112; Sun, 14 May 2023 13:12:24 -0700 (PDT) Message-ID: <2715d1d6-30f5-f472-f963-9c2a57c155b4@gmail.com> Date: Sun, 14 May 2023 23:12:22 +0300 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Firefox/102.0 Thunderbird/102.10.0 To: make-initrd@lists.altlinux.org References: <6b444ed922286eb3df8f5322b1bddf9c55753eb8.1683200226.git.gladkov.alexey@gmail.com> Content-Language: ru, en-US From: Leonid Krivoshein In-Reply-To: <6b444ed922286eb3df8f5322b1bddf9c55753eb8.1683200226.git.gladkov.alexey@gmail.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit Subject: Re: [make-initrd] [PATCH 1/3] Reimplement ueventd X-BeenThere: make-initrd@lists.altlinux.org X-Mailman-Version: 2.1.12 Precedence: list Reply-To: make-initrd@lists.altlinux.org List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 14 May 2023 20:12:27 -0000 Archived-At: List-Archive: Привет! On 5/4/23 16:42, Alexey Gladkov wrote: > [...] > diff --git a/datasrc/ueventd/ueventd.h b/datasrc/ueventd/ueventd.h > new file mode 100644 > index 00000000..a5e50379 > --- /dev/null > +++ b/datasrc/ueventd/ueventd.h К этому заголовочному файлу возникла пара вопросов. > @@ -0,0 +1,76 @@ > +/* SPDX-License-Identifier: GPL-2.0-or-later */ > + > +#ifndef __UEVENTD_H__ > +#define __UEVENTD_H__ > + > +#include > +#include > + > [...] > + > +/* path.c */ > +#include > + > [...] > + > +/* process.c */ > +extern pid_t waitpid_retry(pid_t pid, int *wstatus, int options); > + > +#include 1) Что повторяется дважды, конечно не проблема. Но вообще это необычно выглядит, когда заголовочные файлы упоминаются в чередовании с остальным кодом. Обычно они все выносятся наверх, потому и заголовочные. Такой подход убережёт от понимания ошибок компилятора при появлении в дальнейшем конфликтов в именах между твоим и библиотечным кодом. > + > +extern int is_dot_dir(struct dirent *ent) __attribute__((nonnull(1))); > + > +/* logging.c */ > +#include > +#include > +#include > + > [...] > + > +#endif /* __UEVENTD_H__ */ 2) Понятно, что код чисто Си-шный и никому пока "снаружи" (ueventd) не требуется. Просто, опять же, привычно видеть тут совместимость с "плюсовой обёрткой". Возможно, так задумано, чтобы слинковаться с твоим кодом можно было только для Си-шных программ. -- WBR, Leonid Krivoshein.