remove comments

Author: AvicennaJr

object/strings.go

@@ -96,14 +96,12 @@ func (s *String) format(args []Object) Object {
 	return &String{Value: value}
 }
 
-// Format the string like "Hello {0}", jina ...
 func formatStr(format string, options []Object) (string, error) {
 	var str strings.Builder
 	var val strings.Builder
 	var check_val bool
 	var opts_len int = len(options)
 
-	// This is to enable escaping the {} braces if you want them included
 	var escapeChar bool
 
 	type optM struct {
@@ -113,13 +111,10 @@ func formatStr(format string, options []Object) (string, error) {
 
 	var optionsMap = make(map[int]optM, opts_len)
 
-	// convert the options into a map (may not be the most efficient but we are not going fast)
 	for i, optm := range options {
 		optionsMap[i] = optM{val: false, obj: optm}
 	}
 
-	// Now go through the format string and do the replacement(s)
-	// this has approx time complexity of O(n) (bestest case)
 	for _, opt := range format {
 
 		if !escapeChar && opt == '\\' {
@@ -140,12 +135,9 @@ func formatStr(format string, options []Object) (string, error) {
 		}
 
 		if check_val && opt == '}' {
-			vstr := strings.TrimSpace(val.String()) // remove accidental spaces
-
-			// check the value and try to convert to int
+			vstr := strings.TrimSpace(val.String())
 			arrv, err := strconv.Atoi(vstr)
 			if err != nil {
-				// return non-cryptic go errors
 				return "", fmt.Errorf(fmt.Sprintf("Ulichopeana si NAMBA, jaribu tena: `%s'", vstr))
 			}
 
@@ -156,7 +148,6 @@ func formatStr(format string, options []Object) (string, error) {
 			}
 
 			str.WriteString(oVal.obj.Inspect())
-			// may be innefficient
 			optionsMap[arrv] = optM{val: true, obj: oVal.obj}
 
 			check_val = false
@@ -172,25 +163,15 @@ func formatStr(format string, options []Object) (string, error) {
 		str.WriteRune(opt)
 	}
 
-	// A check if they never closed the formatting braces e.g '{0'
 	if check_val {
 		return "", fmt.Errorf(fmt.Sprintf("Haukufunga '{', tuliokota kabla ya kufika mwisho `%s'", val.String()))
 	}
 
-	// Another innefficient loop
 	for _, v := range optionsMap {
 		if !v.val {
 			return "", fmt.Errorf(fmt.Sprintf("Ulipeana hili chaguo (%s) {%s} lakini haukutumia", v.obj.Inspect(), v.obj.Type()))
 		}
 	}
 
-	// !start
-	// Here we can talk about wtf just happened
-	// 3 loops to do just formatting, formatting for codes sake.
-	// it can be done in 2 loops, the last one is just to confirm that you didn't forget anything.
-	// this is not required but a nice syntatic sugar, we are not here for speed, so ergonomics matter instead of speed
-	// finally we can say we are slower than python!, what an achievement
-	// done!
-
 	return str.String(), nil
 }