Coccinelle archive on
 help / color / Atom feed
From: Markus Elfring <>
To: Nick Desaulniers <>,
	Joe Perches <>,
Cc: Kees Cook <>,
	Alexander Shishkin <>,
	LKML <>,
	Steven Rostedt <>,
	Miguel Ojeda <>,
	Andrew Morton <>,
	Andy Shevchenko <>,
	Coccinelle <>
Subject: Re: [Cocci] string.h: Mark 34 functions with __must_check
Date: Wed, 9 Oct 2019 20:06:15 +0200
Message-ID: <> (raw)
In-Reply-To: <>

> I reviewed the functions here and believe the ones you added checks
> for all look good.

Thanks for your positive feedback.

> Though Joe's comment on the relative order of where the
> annotation appears in the function declarations should be addressed in
> a V2 IMO.

Would you be looking for a subsequent change also by the means of
the semantic patch language with which the (function) attributes
can be adjusted to the ordering that you would prefer finally?

Cocci mailing list

       reply index

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <>
     [not found] ` <20191009110943.7ff3a08a@gandalf.local.home>
     [not found]   ` <>
     [not found]     ` <20191009122735.17415f9c@gandalf.local.home>
     [not found]       ` <>
     [not found]         ` <>
     [not found]           ` <>
2019-10-09 18:06             ` Markus Elfring [this message]
     [not found] ` <>
2019-10-11  5:15   ` [Cocci] Searching for missing variable checks Markus Elfring

Reply instructions:

You may reply publically to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \ \ \ \ \ \ \ \ \ \ \ \ \ \

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

Coccinelle archive on

Archives are clonable:
	git clone --mirror cocci/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 cocci cocci/ \
	public-inbox-index cocci

Example config snippet for mirrors

Newsgroup available over NNTP:

AGPL code for this site: git clone