commit 55014bad62e03ebd1e02cd7f12bdcc48780bcff9
author Steven Mitcham <smitcham@blizzard.com> Wed May 22 13:30:37 2024 -0500
committer Jens Geyer <jensg@apache.org> Thu May 23 21:56:42 2024 +0200
tree 4d3b9352e0db7ef36d942f686a3f7e61788140d9
parent b429cc9fa197b9da45f98f14efcf03bf84f02307

THRIFT-5787 restoring binary compatibility of Factory constructor
Client: netstd
Patch: Steven Mitcham

This closes #2979

lib/netstd/Thrift/Protocol/TBinaryProtocol.cs

diff --git a/lib/netstd/Thrift/Protocol/TBinaryProtocol.cs b/lib/netstd/Thrift/Protocol/TBinaryProtocol.cs
index 6389184..186d220 100644
--- a/lib/netstd/Thrift/Protocol/TBinaryProtocol.cs
+++ b/lib/netstd/Thrift/Protocol/TBinaryProtocol.cs
@@ -475,7 +475,13 @@
             protected readonly bool StrictRead;
             protected readonly bool StrictWrite;
 
-            public Factory(bool strictRead = false, bool strictWrite = true)
+            // emtpy default CTOR required
+            public Factory()
+                : this(false, true)
+            {
+            }
+
+            public Factory(bool strictRead, bool strictWrite)
             {
                 StrictRead = strictRead;
                 StrictWrite = strictWrite;