Skip to content

[lldb] Rename files ValueObjectSyntheticFilter to ValueObjectSynthetic (NFC) #146784

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

kastiglione
Copy link
Contributor

@kastiglione kastiglione commented Jul 2, 2025

Change the name of ValueObjectSyntheticFilter.{h,cpp} to match the main type they declare and define: ValueObjectSynthetic.

@kastiglione kastiglione requested a review from JDevlieghere as a code owner July 2, 2025 21:16
@llvmbot llvmbot added the lldb label Jul 2, 2025
@kastiglione kastiglione changed the title [lldb] Rename files: ValueObjectSyntheticFilter -> ValueObjectSynthetic [lldb] Rename files: ValueObjectSyntheticFilter -> ValueObjectSynthetic (NFC) Jul 2, 2025
@kastiglione kastiglione changed the title [lldb] Rename files: ValueObjectSyntheticFilter -> ValueObjectSynthetic (NFC) [lldb] Rename ValueObjectSyntheticFilter to ValueObjectSynthetic (NFC) Jul 2, 2025
@kastiglione kastiglione changed the title [lldb] Rename ValueObjectSyntheticFilter to ValueObjectSynthetic (NFC) [lldb] Rename files ValueObjectSyntheticFilter to ValueObjectSynthetic (NFC) Jul 2, 2025
@llvmbot
Copy link
Member

llvmbot commented Jul 2, 2025

@llvm/pr-subscribers-lldb

Author: Dave Lee (kastiglione)

Changes

Full diff: https://github.com/llvm/llvm-project/pull/146784.diff

5 Files Affected:

  • (modified) lldb/include/lldb/DataFormatters/TypeSynthetic.h (+3-3)
  • (renamed) lldb/include/lldb/ValueObject/ValueObjectSynthetic.h (+1-1)
  • (modified) lldb/source/ValueObject/CMakeLists.txt (+1-1)
  • (modified) lldb/source/ValueObject/ValueObject.cpp (+1-1)
  • (renamed) lldb/source/ValueObject/ValueObjectSynthetic.cpp (+2-2)
diff --git a/lldb/include/lldb/DataFormatters/TypeSynthetic.h b/lldb/include/lldb/DataFormatters/TypeSynthetic.h
index 11a4ca2cd8330..b147d66def730 100644
--- a/lldb/include/lldb/DataFormatters/TypeSynthetic.h
+++ b/lldb/include/lldb/DataFormatters/TypeSynthetic.h
@@ -56,10 +56,10 @@ class SyntheticChildrenFrontEnd {
   /// This function is assumed to always succeed and if it fails, the front-end
   /// should know to deal with it in the correct way (most probably, by refusing
   /// to return any children). The return value of \ref Update should actually
-  /// be interpreted as "ValueObjectSyntheticFilter cache is good/bad". If this
+  /// be interpreted as "ValueObjectSynthetic cache is good/bad". If this
   /// function returns \ref lldb::ChildCacheState::eReuse, \ref
-  /// ValueObjectSyntheticFilter is allowed to use the children it fetched
-  /// previously and cached. Otherwise, \ref ValueObjectSyntheticFilter must
+  /// ValueObjectSynthetic is allowed to use the children it fetched
+  /// previously and cached. Otherwise, \ref ValueObjectSynthetic must
   /// throw away its cache, and query again for children.
   virtual lldb::ChildCacheState Update() = 0;
 
diff --git a/lldb/include/lldb/ValueObject/ValueObjectSyntheticFilter.h b/lldb/include/lldb/ValueObject/ValueObjectSynthetic.h
similarity index 98%
rename from lldb/include/lldb/ValueObject/ValueObjectSyntheticFilter.h
rename to lldb/include/lldb/ValueObject/ValueObjectSynthetic.h
index cdb2b1ddc3646..063d796ee4eec 100644
--- a/lldb/include/lldb/ValueObject/ValueObjectSyntheticFilter.h
+++ b/lldb/include/lldb/ValueObject/ValueObjectSynthetic.h
@@ -1,4 +1,4 @@
-//===-- ValueObjectSyntheticFilter.h ----------------------------*- C++ -*-===//
+//===-- ValueObjectSynthetic.h ----------------------------*- C++ -*-===//
 //
 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
 // See https://llvm.org/LICENSE.txt for license information.
diff --git a/lldb/source/ValueObject/CMakeLists.txt b/lldb/source/ValueObject/CMakeLists.txt
index db590f757972a..2a61407521bec 100644
--- a/lldb/source/ValueObject/CMakeLists.txt
+++ b/lldb/source/ValueObject/CMakeLists.txt
@@ -14,7 +14,7 @@ add_lldb_library(lldbValueObject
   ValueObjectList.cpp
   ValueObjectMemory.cpp
   ValueObjectRegister.cpp
-  ValueObjectSyntheticFilter.cpp
+  ValueObjectSynthetic.cpp
   ValueObjectUpdater.cpp
   ValueObjectVariable.cpp
   ValueObjectVTable.cpp
diff --git a/lldb/source/ValueObject/ValueObject.cpp b/lldb/source/ValueObject/ValueObject.cpp
index accfd08b30205..a6fb3fad65251 100644
--- a/lldb/source/ValueObject/ValueObject.cpp
+++ b/lldb/source/ValueObject/ValueObject.cpp
@@ -46,7 +46,7 @@
 #include "lldb/ValueObject/ValueObjectConstResult.h"
 #include "lldb/ValueObject/ValueObjectDynamicValue.h"
 #include "lldb/ValueObject/ValueObjectMemory.h"
-#include "lldb/ValueObject/ValueObjectSyntheticFilter.h"
+#include "lldb/ValueObject/ValueObjectSynthetic.h"
 #include "lldb/ValueObject/ValueObjectVTable.h"
 #include "lldb/lldb-private-types.h"
 
diff --git a/lldb/source/ValueObject/ValueObjectSyntheticFilter.cpp b/lldb/source/ValueObject/ValueObjectSynthetic.cpp
similarity index 99%
rename from lldb/source/ValueObject/ValueObjectSyntheticFilter.cpp
rename to lldb/source/ValueObject/ValueObjectSynthetic.cpp
index df774284e4c1b..f673c51a88412 100644
--- a/lldb/source/ValueObject/ValueObjectSyntheticFilter.cpp
+++ b/lldb/source/ValueObject/ValueObjectSynthetic.cpp
@@ -1,4 +1,4 @@
-//===-- ValueObjectSyntheticFilter.cpp ------------------------------------===//
+//===-- ValueObjectSynthetic.cpp ------------------------------------===//
 //
 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
 // See https://llvm.org/LICENSE.txt for license information.
@@ -6,7 +6,7 @@
 //
 //===----------------------------------------------------------------------===//
 
-#include "lldb/ValueObject/ValueObjectSyntheticFilter.h"
+#include "lldb/ValueObject/ValueObjectSynthetic.h"
 
 #include "lldb/Core/Value.h"
 #include "lldb/DataFormatters/TypeSynthetic.h"

@kastiglione kastiglione requested a review from jimingham July 2, 2025 21:16
Copy link
Collaborator

@labath labath left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This has bothered me as well.

Copy link
Collaborator

@jimingham jimingham left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think only Enrico might know why he called it this, and he's not working on lldb anymore.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants