[Lldb-commits] [PATCH] D99867: [lldb] Replace unneeded use of Foundation with ObjectiveC in tests (NFC)

[Dave Lee via Phabricator via lldb-commits]

kastiglione added inline comments.


================
Comment at: lldb/test/API/commands/frame/recognizer/main.m:1
-#import <Foundation/Foundation.h>
+#import <objc/NSObject.h>
 
----------------
teemperor wrote:
> teemperor wrote:
> > I guess this could also be just `stdio.h` (FWIW, the printfs in tests can also be removed as they just require including system headers and are essentially never useful).
> Just to be clear: I'm just saying that removing `printfs` and everything else that references system headers is usually always a good idea but I'm not asking that you should remove them here too. The `stdio.h` is perfect for this patch.
thanks for reviewing closely, a bunch of these changes were automated  and I am guilty of not looking closely (if the test passed then I didn't look closer)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99867