Remove unused stuff

Author: osa1

protobuf/lib/src/protobuf/coded_buffer_writer.dart

@@ -89,14 +89,8 @@ class CodedBufferWriter {
         _writeTag(fieldNumber, WIRETYPE_LENGTH_DELIMITED);
         final mark = _startLengthDelimited();
         _writeValue(
-            PbMap._keyFieldNumber,
-            PbFieldType.toBaseType(fieldValue.keyFieldType),
-            key,
-            keyWireFormat);
-        _writeValue(
-            PbMap._valueFieldNumber,
-            PbFieldType.toBaseType(fieldValue.valueFieldType),
-            value,
+            PbMap._keyFieldNumber, fieldValue.keyFieldType, key, keyWireFormat);
+        _writeValue(PbMap._valueFieldNumber, fieldValue.valueFieldType, value,
             valueWireFormat);
         _endLengthDelimited(mark);
       });

protobuf/lib/src/protobuf/field_type.dart

@@ -144,68 +144,6 @@ enum FieldBaseType {
 
 /// Defines constants and functions for dealing with fieldType bits.
 class PbFieldType {
-  /// Returns the base field type without any of the required, repeated
-  /// and packed bits.
-  static int _baseType(int fieldType) =>
-      fieldType & ~(_REQUIRED_BIT | _REPEATED_BIT | _PACKED_BIT | _MAP_BIT);
-
-  static FieldBaseType toBaseType(int fieldType) {
-    var baseType = fieldType & ~(_REQUIRED_BIT | _REPEATED_BIT | _PACKED_BIT);
-
-    // _MAP sets both message and map bits, probably because in some places we
-    // want to handle maps as messages, but in others we treat them specially..
-    // TODO: figure this out
-    if (baseType == _MAP) {
-      return FieldBaseType.map;
-    }
-
-    assert(baseType & (baseType - 1) == 0,
-        'base type has more than one bit set: $baseType');
-
-    switch (_baseType(fieldType)) {
-      case PbFieldType._BOOL_BIT:
-        return FieldBaseType.bool;
-      case PbFieldType._BYTES_BIT:
-        return FieldBaseType.bytes;
-      case PbFieldType._STRING_BIT:
-        return FieldBaseType.string;
-      case PbFieldType._DOUBLE_BIT:
-        return FieldBaseType.double;
-      case PbFieldType._FLOAT_BIT:
-        return FieldBaseType.float;
-      case PbFieldType._ENUM_BIT:
-        return FieldBaseType.enum_;
-      case PbFieldType._GROUP_BIT:
-        return FieldBaseType.group;
-      case PbFieldType._INT32_BIT:
-        return FieldBaseType.int32;
-      case PbFieldType._INT64_BIT:
-        return FieldBaseType.int64;
-      case PbFieldType._SINT32_BIT:
-        return FieldBaseType.sint32;
-      case PbFieldType._SINT64_BIT:
-        return FieldBaseType.sint64;
-      case PbFieldType._UINT32_BIT:
-        return FieldBaseType.uint32;
-      case PbFieldType._UINT64_BIT:
-        return FieldBaseType.uint64;
-      case PbFieldType._FIXED32_BIT:
-        return FieldBaseType.fixed32;
-      case PbFieldType._FIXED64_BIT:
-        return FieldBaseType.fixed64;
-      case PbFieldType._SFIXED32_BIT:
-        return FieldBaseType.sfixed32;
-      case PbFieldType._SFIXED64_BIT:
-        return FieldBaseType.sfixed64;
-      case PbFieldType._MESSAGE_BIT:
-        return FieldBaseType.message;
-      case PbFieldType._MAP_BIT:
-        return FieldBaseType.map;
-      default:
-        throw ArgumentError('TODO: write an error message');
-    }
-  }
-
   static MakeDefaultFunc? _defaultForType(FieldType type) {
     switch (type.baseType) {
       case FieldBaseType.bool: