[Lldb-commits] [PATCH] D39436: Add regex support to file (-f) and module (-s) breakpoint options.

[Don Hinton via Phabricator via lldb-commits]

hintonda added a comment.

In https://reviews.llvm.org/D39436#911260, @jingham wrote:

> I don't like this change.
>
> First off, the whole point of having options in the commands is so that we don't have to have magic encodings in the values.
>
> We also don't have FileSpec's that resolve to multiple files.  What do "Exists", "IsExecutable",  "GetPath" etc. mean in this context?  There would have to be a really strong reason for making this kind of change.  Just avoiding an extra option does not warrant this change so far as I can see.


It seems that FileSpec already has a dual role, and can be either a search criteria (possibly representing multiple files) or a resolved file.

The former is used with the '-f' and '-s' parameters, and can easily match multiple files, e.g.: if I pass '-f "foo.cpp"', with no directory, it'll match every foo.cpp file included in the target or loaded module.  I'd need to check, but I think 'Exists', 'IsExecutable', and 'GetPath' would still work as expected.


https://reviews.llvm.org/D39436