Re: [PATCH] Script for warning about error leaks

[Julian Foad]
> +#!/bin/sh

> +ATTRIB_RE="__attribute__((warn_unused_result))"
> +ATTRIB_STR="${ATTRIB_RE//\\}" # plain text version of it: remove any backslashes

${foo//bar} is a bashism.

   ATTRIB_STR=$(echo "$ATTRIB_RE" | tr -d '\\')

> +for F in "$@"; do

I'm fairly sure it's portable to drop the in "$@" part:

   for F; do

...though not everyone seems to realise this, so perhaps your version
is clearer in general.

> + else
> + echo "$0: skipping \"$F\": is not a regular file"

Looks a little funny to me; utilities would traditionally just say

       echo "$0: skipping \"$F\": not a regular file"