shithub: mc

Info  •  Files  •  Log  •  Branches

Merge pull request #63 from andrewchambers/memleak

fix small memory leak

--- a/lib/std/fmt.myr

+++ b/lib/std/fmt.myr

@@ -20,6 +20,7 @@

 use "types"

 use "utf"

 use "varargs"

+use "writeall"

 pkg std =

 	/* write to fd */

@@ -103,7 +104,11 @@

 	sb = mksb()

 	sbfmtv(sb, fmt, ap)

 	s = sbfin(sb)

-	-> writeall(fd, s)

+	match writeall(fd, s)

+	| (n, _):

+		slfree(s)

+		-> n

+	;;

 }

 const fmt = {fmt, args

@@ -540,24 +545,6 @@

 		sbputc(sb, b[j - 1])

 	;;

 }

-

-const writeall = {fd, buf

-	var len

-

-	len = 0

-	while true

-		match write(fd, buf[len:])

-		| `Fail err:	break

-		| `Ok n:

-			len += n

-			if n == 0 || len == buf.len

-				break

-			;;

-		;;

-	;;

-	-> len

-}

-

 /* would use std.get(), but that's a dependency loop */

 const getint = {s, msg