[PATCH] D24183: A clang tool for changing surrouding namespaces of class/function definitions.

[Mehdi AMINI via cfe-commits]

mehdi_amini added inline comments.


================
Comment at: clang-tools-extra/trunk/change-namespace/ChangeNamespace.cpp:396
+  // Calculate the name of the `NsDecl` after it is moved to new namespace.
+  std::string OldNs = NsDecl->getQualifiedNameAsString();
+  llvm::StringRef Postfix = OldNs;
----------------
You shouldn't dereference after a `dyn_cast`. Either null-check or use `cast`.


Repository:
  rL LLVM

https://reviews.llvm.org/D24183