odin-lang / Odin

Odin Programming Language
https://odin-lang.org
BSD 3-Clause "New" or "Revised" License
6.55k stars 570 forks source link

strings.make_builder not using given allocator #371

Closed Zilarrezko closed 5 years ago

Zilarrezko commented 5 years ago

Context

https://github.com/odin-lang/Odin/blob/master/core/strings/builder.odin#L11 (core/strings/builder.odin line 11)

dotbmp commented 5 years ago

Easy fix 😄

make_builder :: proc(allocator := context.allocator) -> Builder {
    return Builder{make([dynamic]byte, allocator)};
}
gingerBill commented 5 years ago

Please make a PR!

Zilarrezko commented 5 years ago

Request staged, close if/when accepted.