THRIFT-4831 interface ITProtocolFactory should be class TProtocolFactory again
Client: netstd
Patch: Jens Geyer
This closes #1767
diff --git a/lib/netstd/Thrift/Protocol/TBinaryProtocol.cs b/lib/netstd/Thrift/Protocol/TBinaryProtocol.cs
index 37bca80..7a0243a 100644
--- a/lib/netstd/Thrift/Protocol/TBinaryProtocol.cs
+++ b/lib/netstd/Thrift/Protocol/TBinaryProtocol.cs
@@ -1,4 +1,4 @@
-// Licensed to the Apache Software Foundation(ASF) under one
+// Licensed to the Apache Software Foundation(ASF) under one
// or more contributor license agreements.See the NOTICE file
// distributed with this work for additional information
// regarding copyright ownership.The ASF licenses this file
@@ -588,7 +588,7 @@
return Encoding.UTF8.GetString(buf, 0, buf.Length);
}
- public class Factory : ITProtocolFactory
+ public class Factory : TProtocolFactory
{
protected bool StrictRead;
protected bool StrictWrite;
@@ -604,10 +604,10 @@
StrictWrite = strictWrite;
}
- public TProtocol GetProtocol(TTransport trans)
+ public override TProtocol GetProtocol(TTransport trans)
{
return new TBinaryProtocol(trans, StrictRead, StrictWrite);
}
}
}
-}
\ No newline at end of file
+}
diff --git a/lib/netstd/Thrift/Protocol/TCompactProtocol.cs b/lib/netstd/Thrift/Protocol/TCompactProtocol.cs
index 9ff640a..e6c5dbd 100644
--- a/lib/netstd/Thrift/Protocol/TCompactProtocol.cs
+++ b/lib/netstd/Thrift/Protocol/TCompactProtocol.cs
@@ -1,4 +1,4 @@
-// Licensed to the Apache Software Foundation(ASF) under one
+// Licensed to the Apache Software Foundation(ASF) under one
// or more contributor license agreements.See the NOTICE file
// distributed with this work for additional information
// regarding copyright ownership.The ASF licenses this file
@@ -891,9 +891,9 @@
buf[off + 7] = (byte) ((n >> 56) & 0xff);
}
- public class Factory : ITProtocolFactory
+ public class Factory : TProtocolFactory
{
- public TProtocol GetProtocol(TTransport trans)
+ public override TProtocol GetProtocol(TTransport trans)
{
return new TCompactProtocol(trans);
}
@@ -919,4 +919,4 @@
public const byte Struct = 0x0C;
}
}
-}
\ No newline at end of file
+}
diff --git a/lib/netstd/Thrift/Protocol/TJSONProtocol.cs b/lib/netstd/Thrift/Protocol/TJSONProtocol.cs
index fab1385..8e40c48 100644
--- a/lib/netstd/Thrift/Protocol/TJSONProtocol.cs
+++ b/lib/netstd/Thrift/Protocol/TJSONProtocol.cs
@@ -785,9 +785,9 @@
/// <summary>
/// Factory for JSON protocol objects
/// </summary>
- public class Factory : ITProtocolFactory
+ public class Factory : TProtocolFactory
{
- public TProtocol GetProtocol(TTransport trans)
+ public override TProtocol GetProtocol(TTransport trans)
{
return new TJsonProtocol(trans);
}
diff --git a/lib/netstd/Thrift/Protocol/ITProtocolFactory.cs b/lib/netstd/Thrift/Protocol/TProtocolFactory.cs
similarity index 89%
rename from lib/netstd/Thrift/Protocol/ITProtocolFactory.cs
rename to lib/netstd/Thrift/Protocol/TProtocolFactory.cs
index 3abcbb0..31b0514 100644
--- a/lib/netstd/Thrift/Protocol/ITProtocolFactory.cs
+++ b/lib/netstd/Thrift/Protocol/TProtocolFactory.cs
@@ -20,8 +20,8 @@
namespace Thrift.Protocol
{
// ReSharper disable once InconsistentNaming
- public interface ITProtocolFactory
+ public abstract class TProtocolFactory
{
- TProtocol GetProtocol(TTransport trans);
+ public abstract TProtocol GetProtocol(TTransport trans);
}
-}
\ No newline at end of file
+}