Skip to content

Commit

Permalink
Only set mobile memory limit on supported Go versions
Browse files Browse the repository at this point in the history
  • Loading branch information
neilalexander committed Sep 3, 2023
1 parent 12a3a8c commit c8b9aae
Show file tree
Hide file tree
Showing 3 changed files with 19 additions and 2 deletions.
3 changes: 1 addition & 2 deletions contrib/mobile/mobile.go
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@ import (
"fmt"
"net"
"regexp"
"runtime/debug"

"github.com/gologme/log"

Expand Down Expand Up @@ -45,7 +44,7 @@ func (m *Yggdrasil) StartAutoconfigure() error {
// StartJSON starts a node with the given JSON config. You can get JSON config
// (rather than HJSON) by using the GenerateConfigJSON() function
func (m *Yggdrasil) StartJSON(configjson []byte) error {
debug.SetMemoryLimit(1024 * 1024 * 40)
setMemLimitIfPossible()

m.logger = log.New(m.log, "", 0)
m.logger.EnableLevel("error")
Expand Down
10 changes: 10 additions & 0 deletions contrib/mobile/mobile_mem_go120.go
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
//go:build go1.20
// +build go1.20

package mobile

import "runtime/debug"

func setMemLimitIfPossible() {
debug.SetMemoryLimit(1024 * 1024 * 40)
}
8 changes: 8 additions & 0 deletions contrib/mobile/mobile_mem_other.go
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
//go:build !go1.20
// +build !go1.20

package mobile

func setMemLimitIfPossible() {
// not supported by this Go version
}

0 comments on commit c8b9aae

Please sign in to comment.