Use SignalsPipeUtils::makeCopy util instead of serialize/deserialize

Francois Morin · facebook-github-bot · commit b344177ca114 · 2025-07-07T06:02:51.000-07:00
Summary: Consolidate multiple callers to same Util function

Differential Revision: D77803932

fbshipit-source-id: 2fea7337bd3cfdb661d8360243d220a63d372258
diff --git a/third-party/thrift/src/thrift/lib/hack/src/ThriftImmutableWrapper.php b/third-party/thrift/src/thrift/lib/hack/src/ThriftImmutableWrapper.php
@@ -46,12 +46,7 @@ final public function getInstanceKey(): string {
   }
 
   final public function createDeepCopy(): this::TThrift {
-    return TCompactSerializer::deserialize(
-      TCompactSerializer::serialize($this->data),
-      Classnames::getx($this->data)
-        |> HH\classname_to_class($$)
-        |> $$::withDefaultValues(),
-    );
+    return SignalsPipeUtils::makeCopy($this->data);
   }
 
   final public function serialize(

Original file line number	Diff line number	Diff line change
`@@ -46,12 +46,7 @@ final public function getInstanceKey(): string {`
`46`	`46`	`}`
`47`	`47`
`48`	`48`	`final public function createDeepCopy(): this::TThrift {`
`49`		`- return TCompactSerializer::deserialize(`
`50`		`- TCompactSerializer::serialize($this->data),`
`51`		`- Classnames::getx($this->data)`
`52`		`- \|> HH\classname_to_class($$)`
`53`		`- \|> $$::withDefaultValues(),`
`54`		`- );`
	`49`	`+ return SignalsPipeUtils::makeCopy($this->data);`
`55`	`50`	`}`
`56`	`51`
`57`	`52`	`final public function serialize(`