src/bin/grepc_c: -tuf: Split regex for the function body

Signed-off-by: Alejandro Colomar <alx@kernel.org>

src/bin/grepc_c: -tuf: Fix regex

Fixes: 91c6563acf5e (2025-10-29; "grepc, grepc.1: Add -tu to search for uses")
Signed-off-by: Alejandro Colomar <alx@kernel.org>

src/bin/grepc_c: -tuf: Reuse grepc_c_fld_decl_()

Signed-off-by: Alejandro Colomar <alx@kernel.org>

src/bin/grepc_c: -tfld: Split regex

This will be reused for -tuf.

I copied '[\w\s,;[\]*\?:+-]*\)' from grepc_c_f_params_().

Signed-off-by: Alejandro Colomar <alx@kernel.org>

src/bin/grepc_c: -tfd: Split regex for the function body

This regex part will be reused by other regexes.

Signed-off-by: Alejandro Colomar <alx@kernel.org>

src/bin/grepc_c: -tflp: Simplify implementation

This reuses the regex used by -tfp.

Signed-off-by: Alejandro Colomar <alx@kernel.org>

src/bin/grepc_c: -tf: These can appear in a parameter list: '+', '-', '?', ':'

This adds support for function declarators such as:

int
main(int argc, char *argv[argc + 1]);

char *
seprintf(char dst[], const char end[];
    char dst[dst ? end - dst : 0], const char end[0],
    const char *restrict fmt, ...);

Signed-off-by: Alejandro Colomar <alx@kernel.org>

src/bin/grepc_c: -tf: Use a named capturing group

This allows $identifier to contain capturing groups.

Signed-off-by: Alejandro Colomar <alx@kernel.org>

src/bin/grepc_c: Split regex

The regex for a function declarator is too complex.  Split it into
helper functions that prints the parts of the regex, so that each one
is easier to understand.

Signed-off-by: Alejandro Colomar <alx@kernel.org>

share/mk/: $SHELLCHECKFLAGS: Disable SC2028

It's known that echo(1) doesn't expand anything.  No need to warn.

Signed-off-by: Alejandro Colomar <alx@kernel.org>

src/bin/grepc_*: Use echo(1) instead of printf(1) where possible

echo(1) is fine as long as the first argument doesn't start with a dash.

Signed-off-by: Alejandro Colomar <alx@kernel.org>

src/bin/grepc_c: srcfix (align code)

Signed-off-by: Alejandro Colomar <alx@kernel.org>

src/bin/grepc: -r, -l: Don't color the output of -l

Signed-off-by: Alejandro Colomar <alx@kernel.org>

src/bin/grepc: -r: Simplify implementation, removing a conditional

Signed-off-by: Alejandro Colomar <alx@kernel.org>

src/bin/grepc: -r: Reimplement with grep(1)

With grep(1), we don't need to hard-code escape sequences.

Signed-off-by: Alejandro Colomar <alx@kernel.org>

src/bin/sortman: Fix portability issue

POSIX allows blank characters before '!', but not after.

Reported-by: <onf@disroot.org>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

src/bin/grepc: -x: Simplify implementation

Signed-off-by: Alejandro Colomar <alx@kernel.org>

man/man1/grepc{,_c}.1: Split documentation into a manual page for the driver

I've left grepc_mk undocumented for the moment, as it's still not
as stable as the C driver.

Signed-off-by: Alejandro Colomar <alx@kernel.org>

src/bin/grepc{,_c,_mk}: Align code after split

Signed-off-by: Alejandro Colomar <alx@kernel.org>

src/bin/grepc{,_c,_mk}: Split grepc(1) into helper programs

These programs print the PCRE2 patterns used by grepc(1).  This allows
easily implementing different drivers for parsing different languages.

It also allows easily using pcre2grep(1) in cases where grepc(1) might
be limited; for example, searching in non-regular files, or passing
other flags that grepc(1) doesn't pass to pcre2grep(1).

Signed-off-by: Alejandro Colomar <alx@kernel.org>

src/bin/: Disable pathname expansion

We don't use globs within these scripts, so disable them, for safety
reasons.

Signed-off-by: Alejandro Colomar <alx@kernel.org>

src/bin/grepc: Add missing quotes

Signed-off-by: Alejandro Colomar <alx@kernel.org>

man/man1/grepc.1: SEE ALSO: Fix references

Signed-off-by: Alejandro Colomar <alx@kernel.org>

man/man1/grepc.1: ffix

Signed-off-by: Alejandro Colomar <alx@kernel.org>

src/bin/grepc: Handle xargs(1) errors properly

'xargs grep' is hard, because grep(1) exits with a status of 1 for no
match, which is not an error, but xargs(1) takes that as an error, which
is confused with other errors.

This trick with sh(1) was suggested by Paul, and nicely solves this
problem.

We also need to change $opts to be a normal variable.

We can't export bash array variables, and passing them to sh(1) -c is
not great; escapes could be messed.  It's more robust to use a normal
variable.

Link: <https://lists.gnu.org/archive/html/bug-grep/2025-10/msg00029.html>
Suggested-by: Paul Eggert <eggert@cs.ucla.edu>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

src/bin/grepc: Use the same error code as grep(1)

Exit with 2 on error.

If nothing matches, it's a success, and it exits with 0 normally.
This is different from grep(1), but I don't know how to write
the script to exit with 1 if nothing matches.

This implementation may also ignore some xargs(1) errors.  A future
commit --with code suggested by Paul Eggert-- will fix this.

Link: <https://software.codidact.com/posts/294883>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

src/bin/grepc: Use if/then/fi instead of &&

It will allow using 'set -Eeuo pipefail' and trap(1) ERR.

Signed-off-by: Alejandro Colomar <alx@kernel.org>

man/man1/grepc.1: EXAMPLES: Fix command

The output shows the line number, so it must have been called with -n.

Signed-off-by: Alejandro Colomar <alx@kernel.org>

src/bin/grepc: Refactor to avoid process substitution

Signed-off-by: Alejandro Colomar <alx@kernel.org>

src/bin/grepc: Add defensive check

This is safer than silencing shellcheck(1).

Signed-off-by: Alejandro Colomar <alx@kernel.org>

src/bin/grepc: Add defensive check

In theory, getopts(1) will make sure that no other values arrive here.
However, it's easy to add a check, and that will be safer than silencing
shellcheck(1).

Signed-off-by: Alejandro Colomar <alx@kernel.org>

src/bin/grepc: srcfix (Simplify case/esac)

We don't need ';;&' since the commit referenced below.

Fixes: 0ba68dd03564 (2024-09-19, 2025-10-29; "bin/grepc, grepc.1: -k: Remove flag")
Signed-off-by: Alejandro Colomar <alx@kernel.org>

Adapt grepc(1) to the Linux man-pages project

Signed-off-by: Alejandro Colomar <alx@kernel.org>

man/man1/grepc.1: Use a smaller example

Signed-off-by: Alejandro Colomar <alx@kernel.org>

src/bin/grepc: Use if/else instead of && and ||

Signed-off-by: Alejandro Colomar <alx@kernel.org>

src/bin/grepc: Quote variables

Signed-off-by: Alejandro Colomar <alx@kernel.org>

src/bin/grepc: Ignore shellcheck(1) diagnostics

Signed-off-by: Alejandro Colomar <alx@kernel.org>

src/bin/grepc: Use test(1) instead of [(1)

Signed-off-by: Alejandro Colomar <alx@kernel.org>

man/man1/grepc.1, src/bin/grepc: Add copyright notice

Signed-off-by: Alejandro Colomar <alx@kernel.org>

src/bin/grepc: Handle errors

Signed-off-by: Alejandro Colomar <alx@kernel.org>

man/man1/grepc.1: Adapt to the Linux man-pages project

Signed-off-by: Alejandro Colomar <alx@kernel.org>

man/man1/grepc.1: Move manual page to man/man1/

Signed-off-by: Alejandro Colomar <alx@kernel.org>

src/bin/grepc: Move script to src/bin/

Signed-off-by: Alejandro Colomar <alx@kernel.org>

Import grepc(1) (script and manual page) from separate project

It started as a script in the Linux man-pages project, under scripts/
within scripts/bash_aliases.

Eventually, I turned it into a large script, and moved it into a
separate project, to be able to work on it more freely.

The original scripts were removed from this project in commit
c3752af0549c (2022-11-22; "bash_aliases: Remove grep_syscall(),
grep_syscall_def(), and grep_glibc_prototype()").

Now, grepc(1) has stabilized, and I find it very useful for maintaining
this project (as well as others), so let's merge it back.  This will
allow distros to package it sooner than if it remained as a standalone
project.

I've imported the git history (ignoring files we don't care about) with
the help of git-filter-repo(1).

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc: -tuf: Reuse grepc_c_f_decl_() for the function declarator

It's slightly different than the regex we were using, but I can't see a
reason for it.  It was probably due to forgetting to improve this regex.

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc: -tf: Factor out regex for function declarators

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc: -tt: Add word boundary around the identifier

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc: Add robustness against non-portable pathnames

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc: Use <() instead of $()

Files are simpler to handle.  It results in less flags, and simpler
command invocations.  And it allows breaking the line more nicely,
without having to break in the middle of a quoted region.

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc: -tfp, -tfd: Add support for forward-declarations of parameters

This is currently a (very useful) GNU extension, and soon might become
standard in C2y.

Link: n3394 <https://www.open-std.org/jtc1/sc22/wg14/www/docs/n3394.pdf>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc, grepc.1: -k: Remove flag

Just behave like grep(1) and pcre2grep(1).  If the user wants to see it
more nicely, they can do it themselves.

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc: bugfix: Pass options to grep(1) correctly

Fixes: 92dca8e97bfa ("bin/grepc: srcfix (Variables for options hold their '-')")
Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/, man/: -m: Add flag, and pass it to pcre2grep(1)

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc: -x: c, mk: Add support for Makefile code

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc: Ignore a shellcheck(1) warning

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc: srcfix (Compact cases a little bit)

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc: srcfix (Remove unnecessary comment)

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc: srcfix (Use printf '%s\n' instead of echo)

Reported-by: terdon <https://linux.codidact.com/users/57088>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc: srcfix (Use bash(1) array variable)

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc: srcfix (Remove grepc_search() and main())

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc: srcfix (Remove grepc_patterns())

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc: srcfix (Remove grepc_parse_cmd())

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc: srcfix (Simplify 'case's thanks to bash(1)'s ';;&')

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc: srcfix (Escape '?' in bash(1) case following getopts(1))

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc: srcfix (Compact single-line cases)

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc, grepc.1: -c: Add option to count matches

As in pcre2grep(1) and grep(1).

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc, grepc.1: Rename option -c to -r

This leaves -c unused, so we can add -c with the same meaning of
grep(1): count.

Signed-off-by: Alejandro Colomar <alx@kernel.org>

grepc.1: -c: Remove outdated sentence

Since 83cfbb6042d1 ("bin/grepc: Optimize"), we color also in ther types
of code.  Update the manual page.

Fixes: 83cfbb6042d1 ("bin/grepc: Optimize")
Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc, grepc.1: Use 'l' instead of 's' for Linux system calls

To avoid confusion to users of other kernels.

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc: srcfix (Reorder some function definitions)

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc, grepc.1: -n: Add option to control printing of line numbers

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc: srcfix (Actually create file with mktemp(1))

This ensures we own the file, removing any races.

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc: srcfix (Add some tabs, and remove a function)

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc, grepc.1: -A, -B, -C: Add flags for printing context.

These flags are passed to pcre2grep(1).

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc: srcfix (Variables for options hold their '-')

This is to allow adding options that have arguments that are passed to
pcre2grep(1).

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc, grepc.1: -h: Add option to suppress prefixing file names

Equivalent to grep(1)'s -h, except that we always default to -H behavior.

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc: srcfix (Add missing variable definition)

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc, grepc.1: Remove -h flag

There's already a manual page, and the short help from -h was quite
unhelpful.  I want to reuse -h for grep(1)'s meaning (don't print the
filename).

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc: srcfix (Change variable names)

Use single-letter names for variables that hold command-line options.

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc, grepc.1: Always enable recursive mode; remove -r

The non-recursive behavior is so slow that it's quite useless.  It is
only useful when reading from standard input, but for several files it's
useless.  Make '-r' the default when any files are specified, and remove
the flag (since now it's always enabled).  This happens to bring back a
behavior close to what we had before
2fb22befdfb9 ("Don't find files recursively").

Non-seekable files can still be read, if redirected to standard input.

Update the example from the manual page to reflect current usage.

Signed-off-by: Alejandro Colomar <alx@kernel.org>

Add recursive search back in with -r

Signed-off-by: Alejandro Colomar <alx@kernel.org>

Allow specifying several files

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc: Don't use pcre2grep(1)'s --label

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc: Simplify (remove grepc_search())

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc: Compact code

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc: Optimize

Call pcre2grep(1) only once, which allows removing named pipes.  This is
an important optimization, and will also allow accepting several file
names in the command line (in a future commit).  The source code is also
significantly simplified.

It has a caveat: this single call to pcre2grep(1) will consume more
resources, and will crash on certain input files.  The workaround is to
restrict the search to just one or a few types of code.  Document this
in the manual page.  This caveat only applies to very specific files,
which so far I've only found in the Linux kernel source tree.

Don't find files recursively

grep only one file (or stdin).

The current implementation doesn't allow filterin standard input, which
I miss some times.  I'm removing this feature now, to be able to rewrite
most of the program in a way that allows filtering stdin, and then will
consider adding back a recursive mode if necessary.

Also, don't remove the two blanks at the start of the output, for
consistency when calling this program via xargs(1), which will become
more common now that it only handles single files.

Signed-off-by: Alejandro Colomar <alx@kernel.org>

Remove '-g' option

This program shouldn't know about file names.  If that is wanted, use
something like `find ... | grep ... | xargs grepc ...`.

Signed-off-by: Alejandro Colomar <alx@kernel.org>

Remove '-x' option

This program shouldn't know about file extensions.  If that is wanted,
use something like `find ... | grep ... | xargs grepc ...`.

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc, grepc.1: -tmf, -tmo: Add options to find only function-like macros, or other macros

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc: Shorten function names

Also, use names consistent with the command-line option arguments.

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc: Shorten variable names

Also, use names consistent with the command-line option arguments.

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc: Simplify (remove grepc_use_func())

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc: Simplify (remove grepc_use_type())

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc: Simplify (remove grepc_type())

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc: Simplify (remove grepc_type_typedef())

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc: Simplify (remove grepc_macro())

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc: Compact one-liner definitions

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc: Simplify (remove grepc_glibc_func_def())

Signed-off-by: Alejandro Colomar <alx@kernel.org>

bin/grepc: Improve error messages

Signed-off-by: Alejandro Colomar <alx@kernel.org>