[coreboot-gerrit] [S] Change in coreboot[master]: util/cbmem: Add REG_NEWLINE flag to fix matching pattern

[Konrad Adamczyk (Code Review)]

Konrad Adamczyk has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/74742 )

Change subject: util/cbmem: Add REG_NEWLINE flag to fix matching pattern
......................................................................

Patch Set 2:

(1 comment)

Commit Message:

https://review.coreboot.org/c/coreboot/+/74742/comment/87a9aa33_11b3c966 
PS1, Line 13: REG_NEWLINE
> The only real issue is that the `.?` introduced may match a newline

Indeed it is, right. However, the question mark and, as a consequence, syntax `.?` was introduced in POSIX Extended Regular Expression, without `REG_EXTENDED` flag it's not a special character (just a question mark). That's why I've pointed it out explicitly.
> Maybe we should change the  [^\n]*  to  .*  for consistency now

Good idea, If you're OK with it I can follow up with the patch to replace `[^\n]*` with `.*`.