mib marked 2 inline comments as done.
mib added inline comments.
================
Comment at: lldb/source/Commands/CommandObjectFrame.cpp:880
auto func =
- RegularExpressionSP(new RegularExpression(m_options.m_function));
+ RegularExpressionSP(new
RegularExpression(m_options.m_symbols.front()));
StackFrameRecognizerManager::AddRecognizer(recognizer_sp, module, func);
----------------
labath wrote:
> mib wrote:
> > labath wrote:
> > > Is there something which ensure that m_symbols contains at least one
> > > element here? (i.e., that we do not silently drop the extra symbols
> > > arguments)
> > Few lines above, we make sure the symbols list is not empty.
> Not empty is one thing, but what about the case when it contains 2 or more
> elements (`frame recognizer add --name foo --name bar --regex`)? Will that
> produce some kind of an error, or will it just ignore the second regex ?
It takes only the first symbol and ignores the other ones.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D76188/new/
https://reviews.llvm.org/D76188
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
