From 4ebd4ef72cc74ce805edd9591db7662a47845352 Mon Sep 17 00:00:00 2001
From: zyxkad <zyxkad@gmail.com>
Date: Thu, 29 Feb 2024 22:12:43 -0700
Subject: [PATCH] improve Analyzer forEach error

---
 analyzer.go            | 46 +++++++++++++++++++++++++++++++++++++++---
 cmd/mcla_wasm/errdb.go |  2 +-
 2 files changed, 44 insertions(+), 4 deletions(-)

diff --git a/analyzer.go b/analyzer.go
index a7ceec7..ae371f8 100644
--- a/analyzer.go
+++ b/analyzer.go
@@ -5,6 +5,7 @@ import (
 	"errors"
 	"io"
 	"sync"
+	"time"
 )
 
 type SolutionPossibility struct {
@@ -24,6 +25,10 @@ var (
 
 type Analyzer struct {
 	DB ErrorDB
+
+	errMux        sync.RWMutex
+	lastUpdateErr time.Time
+	cachedErrors  []*ErrorDesc
 }
 
 func NewAnalyzer(db ErrorDB) (a *Analyzer) {
@@ -32,10 +37,46 @@ func NewAnalyzer(db ErrorDB) (a *Analyzer) {
 	}
 }
 
+func (a *Analyzer) UpdateErrors() (err error) {
+	a.errMux.Lock()
+	defer a.errMux.Unlock()
+	return a.updateErrorsLocked()
+}
+
+func (a *Analyzer) updateErrorsLocked() (err error) {
+	errors := make([]*ErrorDesc, 0, 64)
+	if err = a.DB.ForEachErrors(func(e *ErrorDesc) error {
+		errors = append(errors, e)
+		return nil
+	}); err != nil {
+		return
+	}
+	a.lastUpdateErr = time.Now()
+	a.cachedErrors = errors
+	return
+}
+
+func (a *Analyzer) getErrors() []*ErrorDesc {
+	{
+		a.errMux.RLock()
+		needUpdate := a.lastUpdateErr.IsZero() || time.Now().After(a.lastUpdateErr.Add(time.Hour))
+		a.errMux.RUnlock()
+		if needUpdate {
+			a.errMux.Lock()
+			if a.lastUpdateErr.IsZero() || time.Now().After(a.lastUpdateErr.Add(time.Hour)) {
+				a.updateErrorsLocked()
+			}
+			a.errMux.Unlock()
+		}
+	}
+	return a.cachedErrors
+}
+
 func (a *Analyzer) DoError(jerr *JavaError) (matched []SolutionPossibility, err error) {
+	errors := a.getErrors()
 	for jerr != nil {
 		epkg, ecls := rsplit(jerr.Class, '.')
-		a.DB.ForEachErrors(func(e *ErrorDesc) (err error) {
+		for _, e := range errors {
 			sol := SolutionPossibility{
 				ErrorDesc: e,
 			}
@@ -62,8 +103,7 @@ func (a *Analyzer) DoError(jerr *JavaError) (matched []SolutionPossibility, err
 			if sol.Match != 0 { // have any matches
 				matched = append(matched, sol)
 			}
-			return
-		})
+		}
 		jerr = jerr.CausedBy
 	}
 	if matched == nil {
diff --git a/cmd/mcla_wasm/errdb.go b/cmd/mcla_wasm/errdb.go
index e9725b4..9715a21 100644
--- a/cmd/mcla_wasm/errdb.go
+++ b/cmd/mcla_wasm/errdb.go
@@ -53,7 +53,7 @@ func (s *JsStorageCache) Clear() {
 				s.storage.Call("removeItem", key)
 			}
 		}
-	}else if keysFn := s.storage.Get("keys"); keysFn.Type() == js.TypeFunction{
+	} else if keysFn := s.storage.Get("keys"); keysFn.Type() == js.TypeFunction {
 		keys, _ := awaitPromise(keysFn.Invoke())
 		if keys.InstanceOf(Array) {
 			leng := keys.Length()