On Thu, Aug 01, 2019 at 04:53:10PM +0200, Martin Liška wrote:
> --- a/contrib/mklog
> +++ b/contrib/mklog
> @@ -40,6 +40,8 @@ from subprocess import Popen, PIPE
>
> me = os.path.basename(sys.argv[0])
>
> +pr_regex = re.compile('\+\/(\/|\*)\s+(PR [^\/]+\/[0-9]+)')
Perhaps narrow it down to only characters that can actually appear in the
components? So PR [a-z+-]+\/[0-9]+ ?
Also, \/(\/|\*) covers C/C++, but doesn't cover e.g. Fortran.
To cover Fortran, it would need to be
\+(\/(\/|\*)|[Cc*!])\s+
instead.
Jakub
