More TODOs done.

- Removed a TODO without change in DescriptorPool.LookupSymbol - the TODOs were around performance, and this is only used during descriptor initialization
- Make the CodedInputStream limits read-only, adding a static factory method for the rare cases when this is useful
- Extracted IDeepCloneable into its own file.

1 files changed, 0 insertions, 22 deletions

diff --git a/csharp/src/Google.Protobuf/IMessage.cs b/csharp/src/Google.Protobuf/IMessage.cs
index 773845ff..d089f946 100644
--- a/csharp/src/Google.Protobuf/IMessage.cs
+++ b/csharp/src/Google.Protobuf/IMessage.cs
@@ -35,8 +35,6 @@ using Google.Protobuf.Reflection;
 

 namespace Google.Protobuf

 {

-    // TODO(jonskeet): Split these interfaces into separate files when we're happy with them.

-

     /// <summary>

     /// Interface for a Protocol Buffers message, supporting

     /// basic operations required for serialization.

@@ -86,24 +84,4 @@ namespace Google.Protobuf
         /// <param name="message">The message to merge with this one. Must not be null.</param>

         void MergeFrom(T message);

     }

-

-    /// <summary>

-    /// Generic interface for a deeply cloneable type.

-    /// </summary>

-    /// <remarks>

-    /// <para>

-    /// All generated messages implement this interface, but so do some non-message types.

-    /// Additionally, due to the type constraint on <c>T</c> in <see cref="IMessage{T}"/>,

-    /// it is simpler to keep this as a separate interface.

-    /// </para>

-    /// </remarks>

-    /// <typeparam name="T">The type itself, returned by the <see cref="Clone"/> method.</typeparam>

-    public interface IDeepCloneable<T>

-    {

-        /// <summary>

-        /// Creates a deep clone of this object.

-        /// </summary>

-        /// <returns>A deep clone of this object.</returns>

-        T Clone();

-    }

 }