codec: nits

Author: ugorji

codec/cbor_test.go

@@ -164,7 +164,7 @@ func TestCborGoldens(t *testing.T) {
 	bh.MapType = testMapStrIntfTyp
 	bh.RawToString = false
 	bh.SignedInteger = false
-	// ch.SkipUnexpectedTags = false // MARKER 2025 - shouldn't be needed
+	ch.SkipUnexpectedTags = false
 
 	// decode test-cbor-goldens.json into a list of []*testCborGolden
 	// for each one,

codec/codec_run_test.go

@@ -4032,7 +4032,8 @@ func testUpdateExts(nhs ...testNameBasicHandle) {
 		// add extensions for time.Time, excl json and binc (which have builtin support)
 		switch nh.n {
 		case "cbor":
-			sx(timeTyp, 1, tUnixTimeIntfExt) // MARKER 2025 (need to test ext here - not depend on builtin support)
+			// explicitly test ext here - not depend on builtin support)
+			sx(timeTyp, 1, tUnixTimeIntfExt)
 		case "msgpack":
 			sx(timeTyp, 1, tTimeBytesExt)
 		case "simple":
@@ -4146,19 +4147,19 @@ func testEqualH(v1, v2 interface{}, h Handle) (err error) {
 					rcopy(src.Field(i), target.Field(i))
 				}
 			}
-		case reflect.String:
-			// MARKER 2025 - need to all use same functions to compare
-			// s := src.String()
-			// if isJson && bh.StringToRaw {
-			// 	s = base64.StdEncoding.EncodeToString(bytesView(s))
-			// 	// dbuf := make([]byte, base64.StdEncoding.DecodedLen(len(s)))
-			// 	// n, err := base64.StdEncoding.Decode(dbuf, bytesView(s))
-			// 	// if err == nil {
-			// 	// 	s = stringView(dbuf[:n])
-			// 	// }
-			// }
-			// target.SetString(s)
-			target.Set(src)
+		// case reflect.String:
+		// 	// MARKER 2025 - need to all use same functions to compare
+		// 	// s := src.String()
+		// 	// if isJson && bh.StringToRaw {
+		// 	// 	s = base64.StdEncoding.EncodeToString(bytesView(s))
+		// 	// 	// dbuf := make([]byte, base64.StdEncoding.DecodedLen(len(s)))
+		// 	// 	// n, err := base64.StdEncoding.Decode(dbuf, bytesView(s))
+		// 	// 	// if err == nil {
+		// 	// 	// 	s = stringView(dbuf[:n])
+		// 	// 	// }
+		// 	// }
+		// 	// target.SetString(s)
+		// 	target.Set(src)
 		default: // Basic types: int, string, bool, etc.
 			target.Set(src)
 			// if src.Type().AssignableTo(target.Type()) {

codec/encode.go

@@ -1678,15 +1678,15 @@ func (dh helperEncDriver[T]) encFnLoad(rt reflect.Type, rtid uintptr, tinfos *Ty
 // func (e *encoder[T]) encodeIR(iv interface{}, rv reflect.Value) {
 // 	// MARKER: a switch with only concrete types can be optimized.
 // 	// consequently, we deal with nil and interfaces outside the switch.
-
+//
 // 	_ = e.e // early asserts e, e.e are not nil once
 // 	if iv == nil {
 // 		e.e.EncodeNil()
 // 		return
 // 	}
-
+//
 // 	k := rv.Kind()
-
+//
 // 	if isnilBitset.isset(byte(k)) && rvIsNil(rv) {
 // 		if e.h.NilCollectionToZeroLength {
 // 			switch k {
@@ -1701,21 +1701,21 @@ func (dh helperEncDriver[T]) encFnLoad(rt reflect.Type, rtid uintptr, tinfos *Ty
 // 		e.e.EncodeNil()
 // 		return
 // 	}
-
+//
 // 	// rv, isnil := isNil(iv, true) // fast isNil check (e.g. for nil intf or nil pointers)
 // 	// if isnil {
-// 	// 	e.e.EncodeNil() // problem here MARKER 2025
+// 	// 	e.e.EncodeNil() // problem here
 // 	// 	return
 // 	// }
-
+//
 // 	switch v := iv.(type) {
 // 	// case nil:
 // 	// case Selfer:
 // 	case Raw:
 // 		e.rawBytes(v)
 // 	case reflect.Value:
 // 		e.encodeValue(v, nil)
-
+//
 // 	case string:
 // 		e.e.EncodeString(v)
 // 	case bool: