Re: [PATCH v3 2/2 RESEND] checkpatch: warn on empty rust doc comments
Patrick Miller <[email protected]> Wed, 02 Oct 2024 02:17:08 +0000
| Newsgroups | gmane.linux.kernel.rust,gmane.linux.kernel,gmane.linux.newbie |
|---|---|
| Message-ID | <t5SyWXgKghM_fSdMQNH3tTuqZKJck9qMTeUJUWzCWphs_cc7cocnj1gO5cCh0pzzVTCQbEBRqOsYC0om9JIOatBKU3AcoEyFwySAK6B1_Cw=@proton.me> |
This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --------6e9c0b4724622ca2873f2dc07faaadd38d4565d60e5b04cb49ae2ce015d54162 Content-Type: multipart/mixed;boundary=---------------------808725a943ee40242c6bec3ccc1f694e -----------------------808725a943ee40242c6bec3ccc1f694e Content-Transfer-Encoding: quoted-printable Content-Type: text/plain;charset=utf-8 On Monday, September 30th, 2024 at 6:34 AM, Hridesh MG <[email protected]= om> wrote: > = > = > On Mon, Sep 30, 2024 at 8:11=E2=80=AFAM Patrick Miller paddymills@proton= .me wrote: > = > > After the latest revision of my patch 1, I split the if statement so t= hat > > there is a parent check for rust files for future rust patch checks. S= o, > > this would perfectly fit within that block. > > = > > Do you want me to add your code and credit you in my patch? > = > = > Please disregard my prior email, I had failed to CC everyone. Since > the change is small I'm okay with adding it as part of your patch, > please do add the Co-developed-by tag, thanks! > = > However, I was curious how conflicts like these are generally > resolved. For example, if there are two large patchsets which conflict > with each other, how does one ensure that they are compatible, and > even if they are, how do the maintainers ensure that they are applied > in the correct order? 2 changes that I am making to your patch as I merge it with mine (I tested= these) - Added a @fix option. I ran into this with my patch and was requested t= o add it by a checkpatch maintainer - Revised your $prevrawline regex to check against existing blank lines = as well as added blank lines (made the leading + optional). Otherwise I think the checkpatch would not match against a blank doc comment lin= e added after an existing blank doc line. -----------------------808725a943ee40242c6bec3ccc1f694e-- --------6e9c0b4724622ca2873f2dc07faaadd38d4565d60e5b04cb49ae2ce015d54162 Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: ProtonMail wnUEARYKACcFgmb8rRoJkJtRGrsur54RFiEE3KdIkSvUnCve801mm1Eauy6v nhEAAMF4AQCLWQUOtp5+mg3j/o0JBoheDulcy8I2mrzVF4W5OOon7wEAyCOW 8Mi39DzrVpDEkeJEblr54UfE2ssXM7l+bTbw1g4= =Z2AR -----END PGP SIGNATURE----- --------6e9c0b4724622ca2873f2dc07faaadd38d4565d60e5b04cb49ae2ce015d54162--