[Lldb-commits] [PATCH] D151765: [lldb] Introduce the FileSpecBuilder abstraction

[Jim Ingham via Phabricator via lldb-commits]

jingham added a comment.

Why did you choose to have a separate FileSpecBuilder class, rather than making FileSpec smarter about the structure of the path (e.g. have an array of StringRef's into the paths for each component.)   We could do the parse once the first time a path element was requested, and then operations like "RemovePathComponent" would be trivial.  We could maybe even get rid of the distinction between "path" and "filename" since "filename" is really just "last path component".


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D151765/new/

https://reviews.llvm.org/D151765