Added comments

Author: papafe

src/MongoDB.Bson/Serialization/Serializers/BsonDocumentSerializer.cs

@@ -89,7 +89,7 @@ public bool GetDocumentId(
             if (bsonDocument.TryGetValue("_id", out idBsonValue))
             {
                 id = idBsonValue;
-                idGenerator = BsonSerializer.LookupIdGenerator(id.GetType());  //TODO For this we need add a new GetDocumentId to IBsonIdProvider
+                idGenerator = BsonSerializer.LookupIdGenerator(id.GetType());  //TODO ??
 
                 if (idGenerator == null)
                 {

src/MongoDB.Bson/Serialization/Serializers/DictionarySerializerBase.cs

@@ -58,8 +58,7 @@ public DictionarySerializerBase()
         /// </summary>
         /// <param name="dictionaryRepresentation">The dictionary representation.</param>
         public DictionarySerializerBase(DictionaryRepresentation dictionaryRepresentation)
-            : this(dictionaryRepresentation, BsonSerializer.LookupSerializer<object>(), BsonSerializer.LookupSerializer<object>())
-        //TODO We could keep this, and add a new constructor DictionarySerializerBase(dictionaryRepresentation, domain)
+            : this(dictionaryRepresentation, BsonSerializer.LookupSerializer<object>(), BsonSerializer.LookupSerializer<object>())  //TODO We can keep this as is
         {
         }
 
@@ -387,8 +386,7 @@ public DictionarySerializerBase()
         /// </summary>
         /// <param name="dictionaryRepresentation">The dictionary representation.</param>
         public DictionarySerializerBase(DictionaryRepresentation dictionaryRepresentation)
-            : this(dictionaryRepresentation, BsonSerializer.SerializerRegistry)
-        //TODO We could keep this, and add a new constructor DictionarySerializerBase(dictionaryRepresentation, domain)
+            : this(dictionaryRepresentation, BsonSerializer.SerializerRegistry) //TODO We can keep this as is
         {
         }
 

src/MongoDB.Bson/Serialization/Serializers/DiscriminatedInterfaceSerializer.cs

@@ -47,7 +47,7 @@ private static IBsonSerializer<TInterface> CreateInterfaceSerializer()
             var classMapType = classMapDefinition.MakeGenericType(typeof(TInterface));
             var classMap = (BsonClassMap)Activator.CreateInstance(classMapType);
             classMap.AutoMap();
-            classMap.SetDiscriminatorConvention(BsonSerializer.LookupDiscriminatorConvention(typeof(TInterface)));  //TODO This is only called by the constructor, we need a new one with the domain as input
+            classMap.SetDiscriminatorConvention(BsonSerializer.LookupDiscriminatorConvention(typeof(TInterface)));  //TODO ??
             classMap.Freeze();
             return new BsonClassMapSerializer<TInterface>(classMap);
         }
@@ -97,7 +97,7 @@ public DiscriminatedInterfaceSerializer(IDiscriminatorConvention discriminatorCo
 
             _interfaceType = typeof(TInterface);
             _discriminatorConvention = discriminatorConvention ?? interfaceSerializer.GetDiscriminatorConvention();
-            _objectSerializer = BsonSerializer.LookupSerializer<object>();  //TODO We need a new constructor, that takes the domain as input.
+            _objectSerializer = BsonSerializer.LookupSerializer<object>();  //TODO ??
             if (_objectSerializer is ObjectSerializer standardObjectSerializer)
             {
                 _objectSerializer = standardObjectSerializer.WithDiscriminatorConvention(_discriminatorConvention);

src/MongoDB.Bson/Serialization/Serializers/EnumerableSerializerBase.cs

@@ -35,7 +35,7 @@ public abstract class EnumerableSerializerBase<TValue> : SerializerBase<TValue>,
         /// Initializes a new instance of the <see cref="EnumerableSerializerBase{TValue}"/> class.
         /// </summary>
         protected EnumerableSerializerBase()
-            : this(BsonSerializer.SerializerRegistry)  //TODO Need a new constructor that takes the domain as input
+            : this(BsonSerializer.SerializerRegistry) //TODO We can keep this as is
         {
         }
 
@@ -234,7 +234,7 @@ public abstract class EnumerableSerializerBase<TValue, TItem> : SerializerBase<T
         /// Initializes a new instance of the <see cref="EnumerableSerializerBase{TValue, TItem}"/> class.
         /// </summary>
         protected EnumerableSerializerBase()
-            : this(BsonSerializer.SerializerRegistry)   //TODO Need a new constructor that takes the domain as input
+            : this(BsonSerializer.SerializerRegistry) //TODO We can keep this as is
         {
         }
 

src/MongoDB.Bson/Serialization/Serializers/IEnumerableDeserializingAsCollectionSerializer.cs

@@ -50,7 +50,7 @@ private static void EnsureTIEnumerableIsAnInterface()
         /// Initializes a new instance of the IEnumerableDeserializingAsCollectionSerializer class.
         /// </summary>
         public IEnumerableDeserializingAsCollectionSerializer()
-            : this(BsonSerializer.SerializerRegistry)   //TODO Need a new constructor that takes the domain as input
+            : this(BsonSerializer.SerializerRegistry) //TODO We can keep this as is
         {
         }
 

src/MongoDB.Bson/Serialization/Serializers/ImpliedImplementationInterfaceSerializer.cs

@@ -52,7 +52,7 @@ public sealed class ImpliedImplementationInterfaceSerializer<TInterface, TImplem
         /// Initializes a new instance of the <see cref="ImpliedImplementationInterfaceSerializer{TInterface, TImplementation}"/> class.
         /// </summary>
         public ImpliedImplementationInterfaceSerializer()
-            : this(BsonSerializer.SerializerRegistry)   //TODO Need a new constructor that takes the domain as input
+            : this(BsonSerializer.SerializerRegistry) //TODO We can keep this as is
         {
         }
 

src/MongoDB.Bson/Serialization/Serializers/KeyValuePairSerializer.cs

@@ -66,7 +66,7 @@ public KeyValuePairSerializer()
         /// </summary>
         /// <param name="representation">The representation.</param>
         public KeyValuePairSerializer(BsonType representation)
-            : this(representation, BsonSerializer.SerializerRegistry)   //TODO Need a new constructor that takes the domain as input
+            : this(representation, BsonSerializer.SerializerRegistry) //TODO We can keep this as is
         {
         }
 

src/MongoDB.Bson/Serialization/Serializers/NullableSerializer.cs

@@ -64,7 +64,7 @@ public sealed class NullableSerializer<T> :
         /// Initializes a new instance of the <see cref="NullableSerializer{T}"/> class.
         /// </summary>
         public NullableSerializer()
-            : this(BsonSerializer.SerializerRegistry)   //TODO Need a new constructor that takes the domain as input
+            : this(BsonSerializer.SerializerRegistry) //TODO We can keep this as is
         {
         }
 

src/MongoDB.Bson/Serialization/Serializers/ObjectSerializer.cs

@@ -67,7 +67,7 @@ public sealed class ObjectSerializer : ClassSerializerBase<object>, IHasDiscrimi
         /// Initializes a new instance of the <see cref="ObjectSerializer"/> class.
         /// </summary>
         public ObjectSerializer()
-            : this(BsonSerializer.LookupDiscriminatorConvention(typeof(object)))   //TODO Need a new constructor that takes the domain as input
+            : this(BsonSerializer.LookupDiscriminatorConvention(typeof(object))) //TODO We can keep this as is
         {
         }
 
@@ -96,7 +96,7 @@ public ObjectSerializer(IDiscriminatorConvention discriminatorConvention, GuidRe
         /// </summary>
         /// <param name="allowedTypes">A delegate that determines what types are allowed.</param>
         public ObjectSerializer(Func<Type, bool> allowedTypes)
-            : this(BsonSerializer.LookupDiscriminatorConvention(typeof(object)), allowedTypes)   //TODO Need a new constructor that takes the domain as input
+            : this(BsonSerializer.LookupDiscriminatorConvention(typeof(object)), allowedTypes) //TODO We can keep this as is
         {
         }
 

src/MongoDB.Bson/Serialization/Serializers/SerializeAsNominalTypeSerializer.cs

@@ -32,7 +32,7 @@ public sealed class SerializeAsNominalTypeSerializer<TActualType, TNominalType>
         /// Initializes a new instance of the <see cref="SerializeAsNominalTypeSerializer{TActualType, TNominalType}"/> class.
         /// </summary>
         public SerializeAsNominalTypeSerializer()
-            : this(BsonSerializer.SerializerRegistry)   //TODO Need a new constructor that takes the domain as input
+            : this(BsonSerializer.SerializerRegistry) //TODO We can keep this as is
         {
         }