Add permissions request, comment out debug print statements.

appropriate.

.gitignore

@@ -2,3 +2,4 @@ cmd/passgo/passgo
 cmd/passgo-gui/passgo-gui
 nohup.out
 *.apk
+PgpConnect.jar

PgpConnect.java

@@ -0,0 +1,296 @@
+package st.wow.git.passgo;
+
+import java.lang.Runnable;
+import java.lang.String;
+import java.util.concurrent.BlockingQueue;
+import java.util.concurrent.LinkedBlockingQueue;
+import android.os.Handler;
+import java.io.InputStream;
+import java.io.ByteArrayInputStream;
+import java.io.ByteArrayOutputStream;
+import java.io.UnsupportedEncodingException;
+import org.openintents.openpgp.OpenPgpError;
+import org.openintents.openpgp.OpenPgpSignatureResult;
+import org.openintents.openpgp.util.OpenPgpApi;
+import org.openintents.openpgp.util.OpenPgpServiceConnection;
+import android.content.Context;
+import android.content.ClipboardManager;
+import android.content.ClipData;
+import android.content.Intent;
+import android.util.Log;
+import android.app.Activity;
+import android.app.Fragment;
+import android.app.FragmentManager;
+import android.app.FragmentTransaction;
+import android.app.PendingIntent;
+import android.content.IntentSender;
+import android.content.IntentSender.OnFinished;
+import android.content.IntentSender.SendIntentException;
+import android.os.Bundle;
+import android.Manifest;
+import android.content.pm.PackageManager;
+
+public class PgpConnect extends Fragment {
+	Activity act;
+	Context ctx;
+	Handler handler;
+	OpenPgpServiceConnection mServiceConnection;
+	ClipboardManager cb;
+	final int PERMISSIONS_REQUEST = 1;
+
+	public PgpConnect(Activity act) {
+		Log.d("gio", "PgpConnect(Activity)");
+		act = act;
+		ctx = act.getApplicationContext();
+		this.handler = new Handler(ctx.getMainLooper());
+
+		final FragmentManager fm = act.getFragmentManager();
+		final Fragment frag = this;
+		handler.post(new Runnable() {
+			public void run() {
+				Log.d("gio", "PgpConnect(): adding fragment");
+				FragmentTransaction ft = fm.beginTransaction();
+				ft.add(frag, "PgpConnect");
+				ft.commitNow();
+
+				if (ctx.checkSelfPermission(Manifest.permission.READ_EXTERNAL_STORAGE) != PackageManager.PERMISSION_GRANTED || ctx.checkSelfPermission(Manifest.permission.WRITE_EXTERNAL_STORAGE) != PackageManager.PERMISSION_GRANTED) {
+					requestPermissions(new String[]{Manifest.permission.READ_EXTERNAL_STORAGE, Manifest.permission.WRITE_EXTERNAL_STORAGE}, PERMISSIONS_REQUEST);
+				}
+			}
+		});
+	}
+
+	@Override public void onAttach(Context ctx) {
+		super.onAttach(ctx);
+		Log.d("gio", "onAttach()");
+		this.ctx = ctx;
+		this.handler = new Handler(ctx.getMainLooper());
+		mServiceConnection = new OpenPgpServiceConnection(ctx, "org.sufficientlysecure.keychain");
+		mServiceConnection.bindToService();
+		cb = (ClipboardManager) ctx.getSystemService(Context.CLIPBOARD_SERVICE);
+		installComplete(this);
+	}
+
+	//For debugging:
+	/*private void printExtras(Intent data) {
+		Bundle bundle = data.getExtras();
+		if (bundle != null) {
+			for (String k : bundle.keySet()) {
+				Log.d("gio", "extra:" + k);
+			}
+		}
+	}*/
+
+	@Override public void onActivityResult(int requestCode, int resultCode, Intent data) {
+		Log.d("gio", "onActivityResult(" + requestCode + "): " + resultCode);
+		super.onActivityResult(requestCode, resultCode, data);
+		//activityResult(requestCode, resultCode);
+		if (resultCode != Activity.RESULT_OK) {
+			Log.d("gio", "onActivityResult: not OK");
+			stringResult(requestCode, null);
+			return;
+		}
+		switch (data.getAction()) {
+			case OpenPgpApi.ACTION_DECRYPT_VERIFY: {
+				Log.d("gio", "action decrypt");
+				_decrypt(data, requestCode);
+				break;
+			}
+			case OpenPgpApi.ACTION_ENCRYPT: {
+				Log.d("gio", "action encrypt");
+				_encrypt(data, requestCode);
+				break;
+			}
+			case OpenPgpApi.ACTION_GET_KEY_IDS:
+				Log.d("gio", "action getid");
+				String[] ids = data.getStringArrayExtra("user_ids");
+
+				if (ids != null && ids.length > 0) {
+					Log.d("gio", "got some IDs");
+					stringResult(requestCode, ids[0]);
+				} else {
+					Log.d("gio", "no ids");
+					stringResult(requestCode, null);
+				}
+				break;
+			default: {
+				Log.d("gio", "some other action");
+			}
+		}
+	}
+
+	public void Clip(byte []dat) {
+		ClipData clip = ClipData.newPlainText("PgpConnect", new String(dat));
+		cb.setPrimaryClip(clip);
+	}
+
+	public void GetId(int chint) {
+		if (handler == null) {
+			stringResult(chint, null);
+			return;
+		}
+		Log.d("gio", "GetId()");
+		handler.post(new Runnable() {
+			public void run() {
+				Intent data = new Intent();
+				data.setAction(OpenPgpApi.ACTION_GET_KEY_IDS);
+				//data.setAction(OpenPgpApi.ACTION_ENCRYPT);
+				data.putExtra(OpenPgpApi.EXTRA_USER_IDS, new String[]{" "});
+				_getid(data, chint);
+			}
+		});
+	}
+
+	public void Decrypt(byte []dat, int chint) {
+		if (handler == null) {
+			stringResult(chint, null);
+			return;
+		}
+		handler.post(new Runnable() {
+			public void run() {
+				Intent data = new Intent();
+				data.setAction(OpenPgpApi.ACTION_DECRYPT_VERIFY);
+				data.putExtra("DATA", dat);
+				_decrypt(data, chint);
+			}
+		});
+	}
+
+	private void _getid(Intent data, int chint) {
+		Log.d("gio","_getid");
+		InputStream is = null;
+		ByteArrayOutputStream os = new ByteArrayOutputStream();
+		OpenPgpApi api = new OpenPgpApi(this.ctx, mServiceConnection.getService());
+		Intent result = api.executeApi(data, is, os);
+		switch (result.getIntExtra(OpenPgpApi.RESULT_CODE, OpenPgpApi.RESULT_CODE_ERROR)) {
+			case OpenPgpApi.RESULT_CODE_SUCCESS: {
+				Log.d("gio","_getid: success");
+				try {
+					String ret = os.toString("UTF-8");
+					//Log.d(OpenPgpApi.TAG, "output: " + ret);
+					stringResult(chint, ret);
+				} catch (UnsupportedEncodingException e) {
+					Log.e("gio", "UnsupportedEncodingException", e);
+					stringResult(chint, null);
+				}
+				break;
+			}
+			case OpenPgpApi.RESULT_CODE_USER_INTERACTION_REQUIRED: {
+				Log.d("gio","_getid: interaction required");
+				PendingIntent pi = result.getParcelableExtra(OpenPgpApi.RESULT_INTENT);
+				try {
+					startIntentSenderForResult(pi.getIntentSender(), chint, null, 0, 0, 0, null);
+				} catch (IntentSender.SendIntentException e) {
+					Log.e("gio", "SendIntentException", e);
+					stringResult(chint, null);
+				}
+				break;
+			}
+			case OpenPgpApi.RESULT_CODE_ERROR: {
+				Log.d("gio","_getid: error");
+				OpenPgpError error = result.getParcelableExtra(OpenPgpApi.RESULT_ERROR);
+				stringResult(chint, null);
+			}
+		}
+	}
+	private void _decrypt(Intent data, int chint) {
+		Log.d("gio","_decrypt");
+		byte []dat = data.getByteArrayExtra("DATA");
+		InputStream is = new ByteArrayInputStream(dat);
+		ByteArrayOutputStream os = new ByteArrayOutputStream();
+		OpenPgpApi api = new OpenPgpApi(this.ctx, mServiceConnection.getService());
+		Intent result = api.executeApi(data, is, os);
+		switch (result.getIntExtra(OpenPgpApi.RESULT_CODE, OpenPgpApi.RESULT_CODE_ERROR)) {
+			case OpenPgpApi.RESULT_CODE_SUCCESS: {
+				try {
+					String ret = os.toString("UTF-8");
+					//Log.d(OpenPgpApi.TAG, "output: " + ret);
+					stringResult(chint, ret);
+				} catch (UnsupportedEncodingException e) {
+					Log.e("gio", "UnsupportedEncodingException", e);
+					stringResult(chint, null);
+				}
+				break;
+			}
+			case OpenPgpApi.RESULT_CODE_USER_INTERACTION_REQUIRED: {
+				PendingIntent pi = result.getParcelableExtra(OpenPgpApi.RESULT_INTENT);
+				try {
+					startIntentSenderForResult(pi.getIntentSender(), chint, null, 0, 0, 0, null);
+				} catch (IntentSender.SendIntentException e) {
+					Log.e("gio", "SendIntentException", e);
+					stringResult(chint, null);
+				}
+				break;
+			}
+			case OpenPgpApi.RESULT_CODE_ERROR: {
+				OpenPgpError error = result.getParcelableExtra(OpenPgpApi.RESULT_ERROR);
+				stringResult(chint, null);
+			}
+		}
+	}
+
+	public void Encrypt(byte[] id, byte[] dat, int chint) {
+		if (handler == null) {
+			stringResult(chint, null);
+			return;
+		}
+		handler.post(new Runnable() {
+			public void run() {
+				Intent data = new Intent();
+				data.setAction(OpenPgpApi.ACTION_ENCRYPT);
+				data.putExtra(OpenPgpApi.EXTRA_USER_IDS, new String[]{new String(id)});
+				data.putExtra(OpenPgpApi.EXTRA_REQUEST_ASCII_ARMOR, true);
+				data.putExtra("DATA", dat);
+				_encrypt(data, chint);
+			}
+		});
+	}
+
+	private void _encrypt(Intent data, int chint) {
+		Log.d("gio","_encrypt");
+		byte []dat = data.getByteArrayExtra("DATA");
+		InputStream is = new ByteArrayInputStream(dat);
+		ByteArrayOutputStream os = new ByteArrayOutputStream();
+		OpenPgpApi api = new OpenPgpApi(this.ctx, mServiceConnection.getService());
+		Intent result = api.executeApi(data, is, os);
+		switch (result.getIntExtra(OpenPgpApi.RESULT_CODE, OpenPgpApi.RESULT_CODE_ERROR)) {
+			case OpenPgpApi.RESULT_CODE_SUCCESS: {
+				try {
+					String ret = os.toString("UTF-8");
+					//Log.d(OpenPgpApi.TAG, "output: " + ret);
+					stringResult(chint, ret);
+				} catch (UnsupportedEncodingException e) {
+					Log.e("gio", "UnsupportedEncodingException", e);
+					stringResult(chint, null);
+				}
+
+				if (result.hasExtra(OpenPgpApi.RESULT_SIGNATURE)) {
+					OpenPgpSignatureResult sigResult
+							= result.getParcelableExtra(OpenPgpApi.RESULT_SIGNATURE);
+				}
+				break;
+			}
+			case OpenPgpApi.RESULT_CODE_USER_INTERACTION_REQUIRED: {
+				PendingIntent pi = result.getParcelableExtra(OpenPgpApi.RESULT_INTENT);
+				try {
+					IntentSender sender = pi.getIntentSender();
+					//Log.d("PgpConnect", "IntentSender:" + sender.toString());
+					startIntentSenderForResult(sender, chint, null, 0, 0, 0, null);
+				} catch (IntentSender.SendIntentException e) {
+					Log.e("gio", "SendIntentException", e);
+					stringResult(chint, null);
+				}
+				break;
+			}
+			case OpenPgpApi.RESULT_CODE_ERROR: {
+				OpenPgpError error = result.getParcelableExtra(OpenPgpApi.RESULT_ERROR);
+				stringResult(chint, null);
+				break;
+			}
+		}
+	}
+
+	static private native void installComplete(PgpConnect p);
+	static private native void activityResult(int requestCode, int resultCode);
+	static private native void stringResult(int requestCode, String result);
+}

cmd/passgo-gui/impl_android.go

@@ -5,27 +5,41 @@ package main
 import (
 	"os"
 
-	"golang.org/x/image/font/gofont/goregular"
+	"git.wow.st/gmp/passgo"
-	"golang.org/x/image/font/sfnt"
+	"gioui.org/app"
+)
+
+var (
+	noidLabelText = "Enter a GPG key ID above"
 )
 
 func init() {
 	log(Info, "Android start")
-	var err error
+	// Use a larger font on Android
-	regular, err = sfnt.Parse(goregular.TTF)
+	fontSize = 24
+}
+
+func initPgp(w *app.Window) {
+	passgo.InitPgp(w)
+}
+
+func getConfDir() (string, error) {
+	ret, err := app.DataDir()
 	if err != nil {
-		log(Fatal, "Cannot parse default font: ", err)
+		log(Error, "Cannot get data directory:", err)
+		return "", err
 	}
-	confDir = app.DataDir()
+	if _, err := os.Stat(ret); os.IsNotExist(err) {
-	if _, err := os.Stat(confDir); os.IsNotExist(err) {
+		err = os.MkdirAll(ret, 0700)
-		err = os.MkdirAll(confDir, 0700)
 		if err != nil {
-			log(Info, "Cannot create configuration directory ", confDir)
+			log(Error, "Cannot create configuration directory ", ret)
-			log(Fatal, err)
+			return "", err
 		} else {
 			log(Info, "Configuration directory created")
+			return ret, nil
 		}
 	} else {
 		log(Info, "Configuration directory found")
+		return ret, nil
 	}
 }

cmd/passgo-gui/impl_darwin.go

@@ -7,11 +7,14 @@ import (
 	"os/user"
 	"path"
 
-	"golang.org/x/image/font/gofont/goregular"
+	"gioui.org/app"
-	"golang.org/x/image/font/sfnt"
 )
 
-func setFont() error {
+var (
+	noidLabelText = "No GPG ids available. Please create a private key or enter a key ID above"
+)
+
+/*func setFont() error {
 	f, err := os.Open("/System/Library/Fonts/AppleSDGothicNeo.ttc")
 	if err != nil {
 		log(Info, "Cannot open system font.")
@@ -29,28 +32,38 @@ func setFont() error {
 		return err
 	}
 	return nil
-}
+}*/
 
 func init() {
-	err := setFont()
+	fontSize = 16
-	if err != nil {
+	//err := setFont()
-		regular, err = sfnt.Parse(goregular.TTF)
+	//if err != nil {
-		if err != nil {
+	//	regular, err = sfnt.Parse(goregular.TTF)
-			log(Fatal, "Cannot parse default font: ", err)
+	//	if err != nil {
-		}
+	//		log(Fatal, "Cannot parse default font: ", err)
-	}
+	//	}
+	//}
+}
+
+func initPgp(w *app.Window) {
+}
+
+func getConfDir() (string, error) {
 	usr, err := user.Current()
 	if err != nil {
-		log(Fatal, "Cannot get current user: ", err)
+		log(Error, "Cannot get current user: ", err)
+		return "", err
 	}
-	confDir = path.Join(usr.HomeDir, ".config/passgo")
+	ret := path.Join(usr.HomeDir, ".config/passgo")
-	if _, err := os.Stat(confDir); os.IsNotExist(err) {
+	if _, err := os.Stat(ret); os.IsNotExist(err) {
-		err = os.MkdirAll(confDir, 0700)
+		err = os.MkdirAll(ret, 0700)
 		if err != nil {
 			log(Info, "Cannot create configuration directory ", confDir)
-			log(Fatal, err)
+			return "", err
 		} else {
 			log(Info, "Configuration directory created")
+			return ret, nil
 		}
 	}
+	return ret, nil
 }

cmd/passgo-gui/main.go

@@ -3,22 +3,27 @@
 package main
 
 import (
+	"fmt"
 	"io/ioutil"
 	"os"
 	"path"
+	"runtime"
+	"runtime/pprof"
 	"strconv"
 	"strings"
 	"sync"
 	"time"
 
-	"gioui.org/ui"
+	"gioui.org/app"
-	"gioui.org/ui/app"
+	"gioui.org/io/key"
-	"gioui.org/ui/key"
+	"gioui.org/io/system"
-	"gioui.org/ui/layout"
+	"gioui.org/layout"
-	"gioui.org/ui/measure"
+	"gioui.org/text"
-	"gioui.org/ui/text"
+	"gioui.org/unit"
+	"gioui.org/widget"
+	"gioui.org/widget/material"
 
-	"golang.org/x/image/font/sfnt"
+	"gioui.org/font/gofont"
 
 	"github.com/fsnotify/fsnotify"
 	"gopkg.in/yaml.v2"
@@ -33,7 +38,31 @@ type conf struct {
 }
 
 func main() {
+	if false { go func() {
+		f, err := os.Create("cpuprofile")
+		if err != nil {
+			fmt.Printf("Can't create CPU profile\n")
+			os.Exit(-1)
+		}
+		fmt.Printf("Starting CPU profile\n")
+		if err := pprof.StartCPUProfile(f); err != nil {
+			fmt.Printf("Can't start CPU profile\n")
+			f.Close()
+			os.Exit(-1)
+		}
+		time.Sleep(time.Second * 10)
+		fmt.Printf("Stopping CPU profile\n")
+		pprof.StopCPUProfile()
+		f.Close()
+		fmt.Printf("CPU profile written\n")
+	}() }
 	var fd *os.File
+	var err error
+	confDir, err = getConfDir()
+	if err != nil {
+		fmt.Printf("Can't get config directory")
+		os.Exit(-1)
+	}
 	confFile := path.Join(confDir, "config.yml")
 	if _, err := os.Stat(confFile); os.IsNotExist(err) {
 		fd, err = os.Create(confFile)
@@ -71,7 +100,10 @@ func main() {
 	chdir = make(chan struct{})
 	passch = make(chan []byte)
 
+	go func() {
+		log(Info,"passgo.Identities()")
 		passgo.Identities()
+	}()
 	go Updater()
 	log(Info, "Staring event loop")
 	go eventLoop()
@@ -80,8 +112,8 @@ func main() {
 }
 
 var (
+	fontSize float32
 	confDir string
-	regular *sfnt.Font
 	Config  conf
 	l       []passgo.Pass
 	mux     sync.Mutex
@@ -90,10 +122,13 @@ var (
 	updated chan struct{}
 	chdir   chan struct{}
 	passch  chan []byte
+	th *material.Theme
 )
 
 func Updater() {
+	//time.Sleep(time.Second * 2)
 	update := func() {
+		fmt.Printf("update()\n")
 		ltmp, err := store.List()
 		if err != nil {
 			log(Info, err)
@@ -138,7 +173,8 @@ func saveConf(fds ...*os.File) {
 	} else {
 		fd, err = os.Create(path.Join(confDir, "config.yml"))
 		if err != nil {
-			log(Fatal, "Cannot open config file: ", err)
+			log(Error, "Config file = ", path.Join(confDir, "config.yml"))
+			log(Fatal, "Cannot open config file: ", err.Error())
 		}
 	}
 	defer fd.Close()
@@ -154,25 +190,25 @@ func saveConf(fds ...*os.File) {
 }
 
 func eventLoop() {
+	gofont.Register()
+	th = material.NewTheme()
+	th.TextSize = unit.Sp(fontSize)
 	w := app.NewWindow(
-		app.WithWidth(ui.Dp(250)),
+		app.Size(unit.Dp(250), unit.Dp(500)),
-		app.WithTitle("passgo"))
+		app.Title("passgo"))
-	q := w.Queue()
+	gtx := &layout.Context{Queue: w.Queue()}
-	var c ui.Config
+	//time.Sleep(time.Second/5)
-	ops := new(ui.Ops)
+
-	var dims layout.Dimensions
-	var cs layout.Constraints
 	var margincs layout.Constraints
-	var faces measure.Faces
+
 	var c1 layout.FlexChild // flex child for title bar
-	face := faces.For(regular, ui.Sp(16))
 
 	sysinset := &layout.Inset{}
-	margin := layout.UniformInset(ui.Dp(10))
+	margin := layout.UniformInset(unit.Dp(10))
 
-	title := &text.Label{Face: face, Text: "passgo"}
+	title := th.Body1("passgo")
 	dotsBtn := &Button{
-		Face:       face,
+		Size:       unit.Sp(fontSize),
 		Label:      "\xe2\x8b\xae",
 		Alignment:  text.Middle,
 		Color:      black,
@@ -185,12 +221,12 @@ func eventLoop() {
 	lst := &layout.List{Axis: layout.Vertical}
 	passBtns := make([]*Button, 0)
 	pathnames := make([]string, 0)
-	copied := &Overlay{Face: face, Text: "copied to clipboard",
+	copied := &Overlay{Size: unit.Sp(fontSize), Text: "copied to clipboard",
 		Color:      black,
 		Background: darkgray,
 		Alignment:  text.Middle,
 	}
-	cleared := &Overlay{Face: face, Text: "clipboard cleared",
+	cleared := &Overlay{Size: unit.Sp(fontSize), Text: "clipboard cleared",
 		Color:      black,
 		Background: darkgray,
 		Alignment:  text.Middle,
@@ -210,7 +246,7 @@ func eventLoop() {
 				z = "/"
 			}
 			passBtns = append(passBtns, &Button{
-				Face:       face,
+				Size:       unit.Sp(fontSize),
 				Label:      strings.Join([]string{s, n, z}, ""),
 				Background: gray,
 			})
@@ -218,11 +254,11 @@ func eventLoop() {
 		}
 		mux.Unlock()
 	}
-	updateBtns()
 
 	idBtns := make([]*Button, 0)
 
 	updateIdBtns := func() {
+		log(Info,"passgo.Identities()")
 		ids, err := passgo.Identities()
 		if err != nil {
 			log(Info, err)
@@ -231,7 +267,7 @@ func eventLoop() {
 		for i, n := range ids {
 			if i >= len(idBtns) {
 				idBtns = append(idBtns, &Button{
-					Face:       face,
+					Size:       unit.Sp(fontSize),
 					Label:      n,
 					Alignment:  text.End,
 					Color:      black,
@@ -245,70 +281,70 @@ func eventLoop() {
 	}
 
 	confBtn := &Button{
-		Face:       face,
+		Size:       unit.Sp(fontSize),
 		Label:      "configure",
 		Alignment:  text.Middle,
 		Color:      black,
 		Background: gray,
 	}
 
-	storeDirLabel := &text.Label{Face: face, Text: "Store directory"}
+	storeDirLabel := th.Label(unit.Sp(fontSize), "Store directory")
-	storeDirEd := &text.Editor{Face: face, SingleLine: true}
+	storeDirEd := &widget.Editor{ SingleLine: true}
 	storeDirEd.SetText(store.Dir)
 	saveBtn := &Button{
-		Face:       face,
+		Size:       unit.Sp(fontSize),
 		Label:      "save",
 		Alignment:  text.End,
 		Color:      black,
 		Background: gray,
 	}
 	backBtn := &Button{
-		Face:       face,
+		Size:       unit.Sp(fontSize),
 		Label:      "back",
 		Alignment:  text.End,
 		Color:      black,
 		Background: gray,
 	}
-	confirmLabel := &text.Label{Face: face, Text: "Password exists. Overwrite?"}
+	confirmLabel := th.Label(unit.Sp(fontSize), "Password exists. Overwrite?")
 	yesBtn := &Button{
-		Face:       face,
+		Size:       unit.Sp(fontSize),
 		Label:      "yes",
 		Alignment:  text.End,
 		Color:      black,
 		Background: gray,
 	}
 
-	promptLabel := &text.Label{Face: face, Text: "passphrase"}
+	promptLabel := th.Label(unit.Sp(fontSize), "passphrase")
-	promptEd := &text.Editor{Face: face, SingleLine: true, Submit: true}
+	promptEd := &widget.Editor{ SingleLine: true, Submit: true }
 	okBtn := &Button{
-		Face:       face,
+		Size:       unit.Sp(fontSize),
 		Label:      "ok",
 		Alignment:  text.End,
 		Color:      black,
 		Background: gray,
 	}
 	plusBtn := &Button{
-		Face:       face,
+		Size:       unit.Sp(fontSize),
 		Label:      "+",
 		Alignment:  text.Middle,
 		Color:      black,
 		Background: gray,
 	}
 
-	insertLabel := &text.Label{Face: face, Text: "Insert"}
+	insertLabel := th.Label(unit.Sp(fontSize), "Insert")
-	passnameLabel := &text.Label{Face: face, Text: "password name:"}
+	passnameLabel := th.Label(unit.Sp(fontSize), "password name:")
-	passnameEd := &text.Editor{Face: face, SingleLine: true, Submit: true}
+	passnameEd := &widget.Editor{ SingleLine: true }
-	passvalLabel := &text.Label{Face: face, Text: "password value:"}
+	passvalLabel := th.Label(unit.Sp(fontSize), "password value:")
-	passvalEd := &text.Editor{Face: face, SingleLine: true, Submit: true}
+	passvalEd := &widget.Editor{ SingleLine: true, Submit: true }
 
-	noidLabel := &text.Label{Face: face, Text: "No GPG ids available. Please create a private key"}
+	noidLabel := th.Label(unit.Sp(fontSize), noidLabelText)
-	idLabel := &text.Label{Face: face, Text: "Select ID"}
+	idLabel := th.Label(unit.Sp(fontSize), "Select ID")
 
 	anim := &time.Ticker{}
 	animating := false
 	animOn := func() {
 		log(Info, "animOn()")
-		anim = time.NewTicker(time.Second / 120)
+		anim = time.NewTicker(time.Second / 30)
 		animating = true
 		w.Invalidate()
 	}
@@ -319,6 +355,7 @@ func eventLoop() {
 	}
 
 	var listPage, idPage, insertPage, confirmPage, confPage, promptPage, page func()
+	_ = idPage
 
 	prompt := func() []byte {
 		page = promptPage
@@ -333,30 +370,29 @@ func eventLoop() {
 		start2 := float64(Config.ClearDelay)
 		fade2a, end := start2+1.5, start2+2.0
 
-		cs = flex.Flexible(1.0)
+		c2 := flex.Flex(gtx, 1.0, func() {
 			mux.Lock()
 			if lst.Dragging() {
-			key.HideInputOp{}.Add(ops)
+				key.HideInputOp{}.Add(gtx.Ops)
 			}
-		var c2 layout.FlexChild
 			switch {
 			case store.Empty:
-			c2 = flex.End(confBtn.Layout(c, q, ops, cs))
+				confBtn.Layout(gtx)
 				if confBtn.Clicked() {
 					log(Info, "Configure")
 					w.Invalidate()
 					page = confPage
 				}
 			case store.Id == "":
-			c2 = flex.End(idLabel.Layout(ops, cs))
+				idLabel.Layout(gtx)
 				w.Invalidate()
 				page = idPage
 			default:
-			for lst.Init(c, q, ops, cs, len(passBtns)); lst.More(); lst.Next() {
+				gtx.Constraints.Width.Min = gtx.Constraints.Width.Max
-				i := lst.Index()
+				lst.Layout(gtx, len(passBtns), func(i int) {
 					btn := passBtns[i]
-				dims = btn.Layout(c, q, ops, lst.Constraints())
+					gtx.Constraints.Width.Min = gtx.Constraints.Width.Max
-				lst.End(dims)
+					btn.Layout(gtx)
 					if btn.Clicked() {
 						log(Info, "Clicked ", btn.Label)
 						// don't block UI thread on decryption attempt
@@ -385,11 +421,11 @@ func eventLoop() {
 							}
 						}(pathnames[i])
 					}
-			}
+				})
-			c2 = flex.End(lst.Layout())
 			}
 			mux.Unlock()
-		flex.Layout(c1, c2)
+		})
+		flex.Layout(gtx, c1, c2)
 		x := time.Since(overlayStart).Seconds()
 		if x >= fade1b && x < start2 && animating {
 			animOff()
@@ -426,11 +462,12 @@ func eventLoop() {
 				overlay.Color = black
 				overlay.Background = darkgray
 			}
-			cs = margincs
+			gtx.Constraints = margincs
-			al := layout.Align{Alignment: layout.SE}
+			al := layout.Align(layout.SE)
-			cs = al.Begin(ops, cs)
+			al.Layout(gtx, func() {
-			cs.Width.Min = cs.Width.Max
+				gtx.Constraints.Width.Min = gtx.Constraints.Width.Max
-			dims = al.End(overlay.Layout(c, ops, cs))
+				overlay.Layout(gtx)
+			})
 		}
 		if x > start2 && x < fade2a {
 			//	animOff()
@@ -440,24 +477,76 @@ func eventLoop() {
 		}
 	}
 
+	updateBtn := &Button{
+		Size:       unit.Sp(fontSize),
+		Label:      "Update",
+		Alignment:  text.Middle,
+		Color:      black,
+		Background: gray,
+	}
+	idEd := &widget.Editor{SingleLine: true, Submit: true}
+	idSubmitBtn := &Button{
+		Size:       unit.Sp(fontSize),
+		Label:      "Submit",
+		Alignment:  text.Middle,
+		Color:      black,
+		Background: gray,
+	}
+
 	idPage = func() {
 		if !animating {
 			animOn()
 		}
-		cs = flex.Rigid()
+		c2 := flex.Rigid(gtx, func() {
-		c2 := flex.End(idLabel.Layout(ops, cs))
+			idLabel.Layout(gtx)
+		})
 		var c3 layout.FlexChild
-		//if len(idBtns) == 0 {
+		var c4 layout.FlexChild
+		var c5 layout.FlexChild
+		var c6 layout.FlexChild
+		var c7 layout.FlexChild
+		if len(idBtns) == 0 {
+			c3 = flex.Rigid(gtx, func() {
+				updateBtn.Layout(gtx)
+			})
+			if updateBtn.Clicked() {
 				updateIdBtns()
-		//}
+				w.Invalidate()
-		cs = flex.Rigid()
+			}
-		if len(idBtns) == 0 { // still zero after update
+			c4 = flex.Rigid(gtx, func() {
-			c3 = flex.End(noidLabel.Layout(ops, cs))
+				th.Editor("id").Layout(gtx, idEd)
-		} else {
+			})
-			for lst.Init(c, q, ops, cs, len(idBtns)); lst.More(); lst.Next() {
+			for _, e := range idEd.Events(gtx) {
-				lst.End(idBtns[lst.Index()].Layout(c, q, ops, lst.Constraints()))
+				switch e.(type) {
+				case widget.SubmitEvent:
+					log(Info, "Submit")
+					store.Id = idEd.Text()
+					page = listPage
+				}
+			}
+			c5 = flex.Rigid(gtx, func() {
+				idSubmitBtn.Layout(gtx)
+			})
+			if idSubmitBtn.Clicked() {
+				store.Id = idEd.Text()
+				page = listPage
+			}
+			c6 = flex.Rigid(gtx, func() {
+				noidLabel.Layout(gtx)
+			})
+		} else {
+			c3 = flex.Rigid(gtx, func() { })
+			c4 = flex.Rigid(gtx, func() { })
+			c5 = flex.Rigid(gtx, func() { })
+			c6 = flex.Rigid(gtx, func() { })
+		}
+		c7 = flex.Rigid(gtx, func() {
+			if len(idBtns) > 0 { // still zero after update
+				for i := 0; i < len(idBtns); i++ {
+					lst.Layout(gtx, len(idBtns), func(i int) {
+						idBtns[i].Layout(gtx)
+					})
 				}
-			c3 = flex.End(lst.Layout())
 				for _, btn := range idBtns {
 					if btn.Clicked() {
 						log(Info, "ID selected: ", btn.Label)
@@ -468,20 +557,23 @@ func eventLoop() {
 					}
 				}
 			}
-		flex.Layout(c1, c2, c3)
+		})
+		flex.Layout(gtx, c1, c2, c3, c4, c5, c6, c7)
 	}
 
 	var insName, insValue string
-	genBtn := &SelButton{SelColor: darkgray}
+	genBtn := &SelButton{SelColor: gray}
-	genBtn.Button = Button{Face: face, Label: "generate", Background: gray}
+	genBtn.Button = Button{Size: unit.Sp(fontSize), Label: "generate"}
-	symBtn := &SelButton{SelColor: gray, Selected: true}
+	symBtn := &SelButton{SelColor: gray}
-	numBtn := &SelButton{SelColor: gray, Selected: true}
+	numBtn := &SelButton{SelColor: gray}
-	symBtn.Button = Button{Face: face, Label: "@", Background: darkgray}
+	symBtn.Button = Button{Size: unit.Sp(fontSize), Label: "@"}
-	numBtn.Button = Button{Face: face, Label: "#", Background: darkgray}
+	numBtn.Button = Button{Size: unit.Sp(fontSize), Label: "#"}
-	lenEd := &text.Editor{Face: face, SingleLine: true, Alignment: text.End}
+	symBtn.Select()
+	numBtn.Select()
+	lenEd := &widget.Editor{ SingleLine: true, Alignment: text.End }
 	lenEd.SetText("15")
-	lBtn := &Button{Face: face, Label: "<", Background: gray}
+	lBtn := &Button{Size: unit.Sp(fontSize), Label: "<", Background: gray}
-	rBtn := &Button{Face: face, Label: ">", Background: gray}
+	rBtn := &Button{Size: unit.Sp(fontSize), Label: ">", Background: gray}
 
 	updatePw := func() {
 		if !genBtn.Selected {
@@ -499,40 +591,49 @@ func eventLoop() {
 		default:
 			gen = rand.Letter
 		}
-		l,_ := strconv.Atoi(lenEd.Text())
+		l, _ := strconv.Atoi(lenEd.Text())
-		pw,_ := rand.Slice(gen,l)
+		pw, _ := rand.Slice(gen, l)
 		passvalEd.SetText(string(pw))
 	}
 
 	insertPage = func() {
-		c2 := flex.End(insertLabel.Layout(ops, flex.Rigid()))
+		c2 := flex.Rigid(gtx, func() { insertLabel.Layout(gtx) })
+		c3 := flex.Rigid(gtx, func() { passnameLabel.Layout(gtx) })
+		c4 := flex.Rigid(gtx, func() { th.Editor("name").Layout(gtx, passnameEd) })
+		c5 := flex.Rigid(gtx, func() { passvalLabel.Layout(gtx) })
+		c6 := flex.Rigid(gtx, func() { th.Editor("password").Layout(gtx, passvalEd) })
 
-		c3 := flex.End(passnameLabel.Layout(ops, flex.Rigid()))
-		c4 := flex.End(passnameEd.Layout(c, q, ops, flex.Rigid()))
-		c5 := flex.End(passvalLabel.Layout(ops, flex.Rigid()))
-		c6 := flex.End(passvalEd.Layout(c, q, ops, flex.Rigid()))
-
-		al := &layout.Align{Alignment: layout.E}
 		btnflx := &layout.Flex{Axis: layout.Horizontal}
-		btnflx.Init(ops, al.Begin(ops, flex.Rigid()))
+		al := layout.Align(layout.E)
-		bc1 := btnflx.End(lBtn.Layout(c, q, ops, btnflx.Rigid()))
+		c7 := flex.Rigid(gtx, func() {
-		cs := btnflx.Rigid()
+			bc1 := btnflx.Rigid(gtx, func() { lBtn.Layout(gtx) })
-		cs.Width.Min = 60
+			bc2 := btnflx.Rigid(gtx, func() {
-		bc2 := btnflx.End(lenEd.Layout(c, q, ops, cs))
+				gtx.Constraints.Width.Min = 60
-		bc3 := btnflx.End(rBtn.Layout(c, q, ops, btnflx.Rigid()))
+				th.Editor("len").Layout(gtx, lenEd)
-		bc4 := btnflx.End(symBtn.Layout(c, q, ops, btnflx.Rigid()))
+			})
-		bc5 := btnflx.End(numBtn.Layout(c, q, ops, btnflx.Rigid()))
-		bc6 := btnflx.End(genBtn.Layout(c, q, ops, btnflx.Rigid()))
-		c7 := flex.End(al.End(btnflx.Layout(bc1, bc2, bc3, bc4, bc5, bc6)))
 
-		btnflx.Init(ops, al.Begin(ops, flex.Rigid()))
+			bc3 := btnflx.Rigid(gtx, func() { rBtn.Layout(gtx) })
-		bc1 = btnflx.End(backBtn.Layout(c, q, ops, btnflx.Rigid()))
+			bc4 := btnflx.Rigid(gtx, func() { symBtn.Layout(gtx) })
-		bc2 = btnflx.End(saveBtn.Layout(c, q, ops, btnflx.Rigid()))
+			bc5 := btnflx.Rigid(gtx, func() { numBtn.Layout(gtx) })
-		c8 := flex.End(al.End(btnflx.Layout(bc1, bc2)))
+			bc6 := btnflx.Rigid(gtx, func() { genBtn.Layout(gtx) })
-		flex.Layout(c1, c2, c3, c4, c5, c6, c7, c8)
+
+			al.Layout(gtx, func() {
+				btnflx.Layout(gtx, bc1, bc2, bc3, bc4, bc5, bc6)
+			})
+		})
+
+		c8 := flex.Rigid(gtx, func() {
+			bc1 := btnflx.Rigid(gtx, func() { backBtn.Layout(gtx) })
+			bc2 := btnflx.Rigid(gtx, func() { saveBtn.Layout(gtx) })
+			al.Layout(gtx, func() {
+				btnflx.Layout(gtx, bc1, bc2)
+			})
+		})
+
+		flex.Layout(gtx, c1, c2, c3, c4, c5, c6, c7, c8)
 
 		if lBtn.Clicked() {
-			l,_ := strconv.Atoi(lenEd.Text())
+			l, _ := strconv.Atoi(lenEd.Text())
 			if l > 0 {
 				l -= 1
 			}
@@ -541,8 +642,8 @@ func eventLoop() {
 			w.Invalidate()
 		}
 		if rBtn.Clicked() {
-			l,_ := strconv.Atoi(lenEd.Text())
+			l, _ := strconv.Atoi(lenEd.Text())
-			lenEd.SetText(strconv.Itoa(l+1))
+			lenEd.SetText(strconv.Itoa(l + 1))
 			updatePw()
 			w.Invalidate()
 		}
@@ -575,20 +676,31 @@ func eventLoop() {
 					return
 				}
 			}
-			store.Insert(passnameEd.Text(), passvalEd.Text())
+			//Do not block the UI thread.
+			go func() {
+				err := store.Insert(passnameEd.Text(), passvalEd.Text())
+				if err != nil {
+					page = idPage
+				}
+			}()
 		}
 	}
 
 	confirmPage = func() {
-		cs = flex.Rigid()
+		c2 := flex.Rigid(gtx, func() {
-		c2 := flex.End(confirmLabel.Layout(ops, cs))
+			confirmLabel.Layout(gtx)
-		al := &layout.Align{Alignment: layout.E}
+		})
+		al := layout.Align(layout.E)
 		btnflx := &layout.Flex{Axis: layout.Horizontal}
-		btnflx.Init(ops, al.Begin(ops, flex.Rigid()))
+		c3 := flex.Rigid(gtx, func() {
-		bc1 := btnflx.End(backBtn.Layout(c, q, ops, btnflx.Rigid()))
+			bc1 := btnflx.Rigid(gtx, func() { backBtn.Layout(gtx) })
-		bc2 := btnflx.End(yesBtn.Layout(c, q, ops, btnflx.Rigid()))
+			bc2 := btnflx.Rigid(gtx, func() { yesBtn.Layout(gtx) })
-		c3 := flex.End(al.End(btnflx.Layout(bc1, bc2)))
+
-		flex.Layout(c1, c2, c3)
+			al.Layout(gtx, func() {
+				btnflx.Layout(gtx, bc1, bc2)
+			})
+		})
+		flex.Layout(gtx, c1, c2, c3)
 
 		if backBtn.Clicked() {
 			w.Invalidate()
@@ -597,27 +709,34 @@ func eventLoop() {
 		if yesBtn.Clicked() {
 			w.Invalidate()
 			page = listPage
-			store.Insert(insName, insValue)
+			go func() {
+				err := store.Insert(insName, insValue)
+				if err != nil {
+					page = idPage
+				}
+			}()
 		}
 	}
 
 	confPage = func() {
-		cs = flex.Rigid()
+		c2 := flex.Rigid(gtx, func() { storeDirLabel.Layout(gtx) })
-		c2 := flex.End(storeDirLabel.Layout(ops, cs))
+		c3 := flex.Rigid(gtx, func() { th.Editor("directory").Layout(gtx, storeDirEd) })
-		cs = flex.Rigid()
+
-		c3 := flex.End(storeDirEd.Layout(c, q, ops, cs))
+		al := layout.Align(layout.E)
-		cs = flex.Rigid()
+		c4 := flex.Rigid(gtx, func() {
-		al := &layout.Align{Alignment: layout.E}
-		cs = al.Begin(ops, cs)
 			btnflx := &layout.Flex{Axis: layout.Horizontal}
-		btnflx.Init(ops, cs)
+			bc1 := btnflx.Rigid(gtx, func() {
-		cs = btnflx.Rigid()
+				backBtn.Layout(gtx)
-		bc1 := btnflx.End(backBtn.Layout(c, q, ops, cs))
+			})
-		cs = btnflx.Rigid()
+			bc2 := btnflx.Rigid(gtx, func() {
-		bc2 := btnflx.End(saveBtn.Layout(c, q, ops, cs))
+				saveBtn.Layout(gtx)
-		dims = btnflx.Layout(bc1, bc2)
+			})
-		c4 := flex.End(al.End(dims))
+			al.Layout(gtx, func() {
-		flex.Layout(c1, c2, c3, c4)
+				btnflx.Layout(gtx, bc1, bc2)
+			})
+		})
+		flex.Layout(gtx, c1, c2, c3, c4)
+
 		if backBtn.Clicked() {
 			log(Info, "Back")
 			storeDirEd.SetText(store.Dir)
@@ -642,29 +761,30 @@ func eventLoop() {
 
 	promptPage = func() {
 		submit := false
-		for e, ok := promptEd.Next(c, q); ok; e, ok = promptEd.Next(c, q) {
+		for _, e := range promptEd.Events(gtx) {
 			switch e.(type) {
-			case text.SubmitEvent:
+			case widget.SubmitEvent:
 				log(Info, "Submit")
 				submit = true
 			}
 		}
-		cs = flex.Rigid()
+		c2 := flex.Rigid(gtx, func() { promptLabel.Layout(gtx) })
-		c2 := flex.End(promptLabel.Layout(ops, cs))
+		c3 := flex.Rigid(gtx, func() { th.Editor("password").Layout(gtx, promptEd) })
-		cs = flex.Rigid()
+		c4 := flex.Rigid(gtx, func() {
-		c3 := flex.End(promptEd.Layout(c, q, ops, cs))
+			al := layout.Align(layout.E)
-		cs = flex.Rigid()
-		al := &layout.Align{Alignment: layout.E}
-		cs = al.Begin(ops, cs)
 			btnflx := &layout.Flex{Axis: layout.Horizontal}
-		btnflx.Init(ops, cs)
+			bc1 := btnflx.Rigid(gtx, func() {
-		cs = btnflx.Rigid()
+				backBtn.Layout(gtx)
-		bc1 := btnflx.End(backBtn.Layout(c, q, ops, cs))
+			})
-		cs = btnflx.Rigid()
+			bc2 := btnflx.Rigid(gtx, func() {
-		bc2 := btnflx.End(okBtn.Layout(c, q, ops, cs))
+				okBtn.Layout(gtx)
-		dims = btnflx.Layout(bc1, bc2)
+			})
-		c4 := flex.End(al.End(dims))
+			al.Layout(gtx, func() {
-		flex.Layout(c1, c2, c3, c4)
+				btnflx.Layout(gtx, bc1, bc2)
+			})
+		})
+		flex.Layout(gtx, c1, c2, c3, c4)
+
 		if submit || okBtn.Clicked() {
 			log(Info, "Ok")
 			go func() { // do not block UI thread
@@ -685,6 +805,9 @@ func eventLoop() {
 
 	page = listPage
 
+	ms := &runtime.MemStats{}
+	x := 0
+	var mallocs uint64
 	for {
 		select {
 		case <-updated:
@@ -694,36 +817,49 @@ func eventLoop() {
 		case <-anim.C:
 			w.Invalidate()
 		case e := <-w.Events():
+			x++
+			if x == 100 {
+				runtime.ReadMemStats(ms)
+				mallocs = ms.Mallocs
+			}
 			switch e := e.(type) {
-			case app.DestroyEvent:
+			case system.DestroyEvent:
 				return
-			case app.UpdateEvent:
+			case system.StageEvent:
-				c = &e.Config
+				if e.Stage == system.StageRunning {
-				ops.Reset()
+					initPgp(w)
-				faces.Reset(c)
+				}
-				cs = layout.RigidConstraints(e.Size)
+				go func() {
+					updateIdBtns()
+				}()
+			case system.FrameEvent:
+				gtx.Reset(e.Config, e.Size)
+
 				sysinset.Top = e.Insets.Top
 				sysinset.Bottom = e.Insets.Bottom
 				sysinset.Left = e.Insets.Left
 				sysinset.Right = e.Insets.Right
-				cs = sysinset.Begin(c, ops, cs)
+
-				cs = margin.Begin(c, ops, cs)
+				sysinset.Layout(gtx, func() {
-				margincs = cs
+					margin.Layout(gtx, func() {
-				flex.Init(ops, cs)
+						margincs = gtx.Constraints
-				cs = flex.Rigid()
+						c1 = flex.Rigid(gtx, func() {
-				titleflex.Init(ops, cs)
+							ct2 := titleflex.Rigid(gtx, func() {
-				cs = titleflex.Rigid()
+								plusBtn.Layout(gtx)
-				ct2 := titleflex.End(plusBtn.Layout(c, q, ops, cs))
+							})
-				cs = titleflex.Rigid()
+							ct3 := titleflex.Rigid(gtx, func() {
-				ct3 := titleflex.End(dotsBtn.Layout(c, q, ops, cs))
+								dotsBtn.Layout(gtx)
-				cs = titleflex.Flexible(1.0)
+							})
-				cs.Width.Min = cs.Width.Max
+							ct1 := titleflex.Flex(gtx, 1.0, func() {
-				ct1 := titleflex.End(title.Layout(ops, cs))
+								gtx.Constraints.Width.Min = gtx.Constraints.Width.Max
-				c1 = flex.End(titleflex.Layout(ct1, ct2, ct3))
+								title.Layout(gtx)
+							})
+							titleflex.Layout(gtx, ct1, ct2, ct3)
+						})
 
 						page()
-
+					})
-				sysinset.End(margin.End(dims))
+				})
 
 				if dotsBtn.Clicked() {
 					log(Info, "Configure")
@@ -736,7 +872,12 @@ func eventLoop() {
 					insName, insValue = "", ""
 					page = insertPage
 				}
-				w.Update(ops)
+				e.Frame(gtx.Ops)
+			}
+			if x == 100 {
+				x = 0
+				runtime.ReadMemStats(ms)
+				fmt.Printf("mallocs: %d\n", ms.Mallocs-mallocs)
 			}
 		}
 	}

cmd/passgo-gui/ui.go

@@ -4,14 +4,14 @@ import (
 	"image"
 	"image/color"
 
-	"gioui.org/ui"
+	"gioui.org/f32"
-	"gioui.org/ui/f32"
+	"gioui.org/gesture"
-	"gioui.org/ui/gesture"
+	"gioui.org/io/pointer"
-	"gioui.org/ui/input"
+	"gioui.org/layout"
-	"gioui.org/ui/layout"
+	"gioui.org/op/clip"
-	"gioui.org/ui/paint"
+	"gioui.org/op/paint"
-	"gioui.org/ui/pointer"
+	"gioui.org/text"
-	"gioui.org/ui/text"
+	"gioui.org/unit"
 )
 
 var (
@@ -22,7 +22,7 @@ var (
 )
 
 type Overlay struct {
-	Face       text.Face
+	Size       unit.Value
 	Text       string
 	Click      gesture.Click
 	Color      color.RGBA
@@ -30,30 +30,24 @@ type Overlay struct {
 	Alignment  text.Alignment
 }
 
-func (b *Overlay) Layout(c ui.Config, ops *ui.Ops, cs layout.Constraints) layout.Dimensions {
+func (b *Overlay) Layout(gtx *layout.Context) {
-	ins := layout.UniformInset(ui.Dp(1))
+	ins := layout.UniformInset(unit.Dp(1))
-	cs = ins.Begin(c, ops, cs)
+	ins.Layout(gtx, func() {
-	var dims layout.Dimensions
 		st := layout.Stack{}
-	st.Init(ops, cs)
+		c2 := st.Rigid(gtx, func() {
-	{
+			l := th.Label(b.Size, b.Text)
-		cs = st.Rigid()
+			ins := layout.UniformInset(unit.Dp(4))
-		l := text.Label{
+			l.Color = b.Color
-			Face:      b.Face,
+			ins.Layout(gtx, func() {
-			Text:      b.Text,
+				l.Layout(gtx)
-			Alignment: b.Alignment,
+			})
-		}
+			pointer.RectAreaOp{image.Rect(0, 0, gtx.Dimensions.Size.X, gtx.Dimensions.Size.Y)}.Add(gtx.Ops)
-		ins := layout.UniformInset(ui.Dp(4))
+		})
-		l.Material.Record(ops)
+		c1 := st.Expand(gtx, func() {
-		paint.ColorOp{Color: b.Color}.Add(ops)
+			layoutRRect(b.Background, gtx)
-		l.Material.Stop()
+		})
-		dims = ins.End(l.Layout(ops, ins.Begin(c, ops, cs)))
+		st.Layout(gtx, c1, c2)
-		pointer.RectAreaOp{image.Rect(0, 0, dims.Size.X, dims.Size.Y)}.Add(ops)
+	})
-	}
-	c2 := st.End(dims)
-	c1 := st.End(layoutRRect(b.Background, c, ops, st.Expand()))
-	dims = st.Layout(c1, c2)
-	return ins.End(dims)
 }
 
 type SelButton struct {
@@ -63,7 +57,7 @@ type SelButton struct {
 }
 
 type Button struct {
-	Face       text.Face
+	Size       unit.Value
 	Label      string
 	Click      gesture.Click
 	Color      color.RGBA
@@ -72,75 +66,73 @@ type Button struct {
 	clicked    bool
 }
 
-func layoutRRect(col color.RGBA, c ui.Config, ops *ui.Ops, cs layout.Constraints) layout.Dimensions {
+func layoutRRect(col color.RGBA, gtx *layout.Context) {
-	r := float32(c.Px(ui.Dp(4)))
+	r := float32(gtx.Config.Px(unit.Dp(4)))
-	sz := image.Point{X: cs.Width.Min, Y: cs.Height.Min}
+	sz := image.Point{X: gtx.Constraints.Width.Min, Y: gtx.Constraints.Height.Min}
 	w, h := float32(sz.X), float32(sz.Y)
-	rrect(ops, w, h, r, r, r, r)
+	rect := f32.Rectangle{
-	paint.ColorOp{Color: col}.Add(ops)
+			f32.Point{0, 0},
-	paint.PaintOp{Rect: f32.Rectangle{Max: f32.Point{X: w, Y: h}}}.Add(ops)
+			f32.Point{w, h},
-	return layout.Dimensions{Size: sz}
+		}
+	clip.RoundRect(gtx.Ops, rect, r, r, r, r)
+	paint.ColorOp{Color: col}.Add(gtx.Ops)
+	paint.PaintOp{Rect: f32.Rectangle{Max: f32.Point{X: w, Y: h}}}.Add(gtx.Ops)
+	gtx.Dimensions = layout.Dimensions{Size: sz}
 }
 
-// https://pomax.github.io/bezierinfo/#circles_cubic.
+func (b *Button) Layout(gtx *layout.Context) {
-func rrect(ops *ui.Ops, width, height, se, sw, nw, ne float32) {
-	w, h := float32(width), float32(height)
-	const c = 0.55228475 // 4*(sqrt(2)-1)/3
-	var b paint.PathBuilder
-	b.Init(ops)
-	b.Move(f32.Point{X: w, Y: h - se})
-	b.Cube(f32.Point{X: 0, Y: se * c}, f32.Point{X: -se + se*c, Y: se}, f32.Point{X: -se, Y: se}) // SE
-	b.Line(f32.Point{X: sw - w + se, Y: 0})
-	b.Cube(f32.Point{X: -sw * c, Y: 0}, f32.Point{X: -sw, Y: -sw + sw*c}, f32.Point{X: -sw, Y: -sw}) // SW
-	b.Line(f32.Point{X: 0, Y: nw - h + sw})
-	b.Cube(f32.Point{X: 0, Y: -nw * c}, f32.Point{X: nw - nw*c, Y: -nw}, f32.Point{X: nw, Y: -nw}) // NW
-	b.Line(f32.Point{X: w - ne - nw, Y: 0})
-	b.Cube(f32.Point{X: ne * c, Y: 0}, f32.Point{X: ne, Y: ne - ne*c}, f32.Point{X: ne, Y: ne}) // NE
-	b.End()
-}
-
-func (b *Button) Layout(c ui.Config, q input.Queue, ops *ui.Ops, cs layout.Constraints) layout.Dimensions {
 	b.clicked = false
-	for ev, ok := b.Click.Next(q); ok; ev, ok = b.Click.Next(q) {
+	for _, ev := range b.Click.Events(gtx) {
 		if ev.Type == gesture.TypeClick {
 			b.clicked = true
 		}
 	}
-	ins := layout.UniformInset(ui.Dp(1))
+	ins := layout.UniformInset(unit.Dp(1))
-	cs = ins.Begin(c, ops, cs)
+	ins.Layout(gtx, func() {
-	var dims layout.Dimensions
 		st := layout.Stack{}
-	st.Init(ops, cs)
+		c2 := st.Rigid(gtx, func() {
-	{
+			l := th.Label(b.Size, b.Label)
-		cs = st.Rigid()
+			ins := layout.UniformInset(unit.Dp(4))
-		l := text.Label{
-			Face:      b.Face,
-			Text:      b.Label,
-			Alignment: b.Alignment,
-		}
-		ins := layout.UniformInset(ui.Dp(4))
 			//paint.ColorOp{Color: b.Color}.Add(ops)
-		dims = ins.End(l.Layout(ops, ins.Begin(c, ops, cs)))
+			ins.Layout(gtx, func() {
-		pointer.RectAreaOp{image.Rect(0, 0, dims.Size.X, dims.Size.Y)}.Add(ops)
+				l.Layout(gtx)
-		b.Click.Add(ops)
+			})
-	}
+			pointer.RectAreaOp{image.Rect(0, 0, gtx.Dimensions.Size.X, gtx.Dimensions.Size.Y)}.Add(gtx.Ops)
-	c2 := st.End(dims)
+			b.Click.Add(gtx.Ops)
-	c1 := st.End(layoutRRect(b.Background, c, ops, st.Expand()))
+		})
-	dims = st.Layout(c1, c2)
+		c1 := st.Expand(gtx, func() {
-	return ins.End(dims)
+			layoutRRect(b.Background, gtx)
+		})
+		st.Layout(gtx, c1, c2)
+	})
 }
 
 func (b *Button) Clicked() bool {
 	return b.clicked
 }
 
-func (b *SelButton) Clicked() bool {
+func (b *SelButton) Toggle() {
-	if b.clicked {
 	b.Selected = !b.Selected
 	b.SelColor, b.Background = b.Background, b.SelColor
+}
+
+func (b *SelButton) Select() {
+	if !b.Selected {
+		b.Toggle()
+	}
+}
+
+func (b *SelButton) Deselect() {
+	if b.Selected {
+		b.Toggle()
+	}
+}
+
+func (b *SelButton) Clicked() bool {
+	if b.clicked {
+		b.Toggle()
 		return true
 	} else {
 		return false
 	}
 }
-

go.mod

@@ -1,14 +0,0 @@
-module git.wow.st/gmp/passgo
-
-go 1.12
-
-require (
-	gioui.org/ui v0.0.0-20190918172808-816f0e901fc6
-	git.wow.st/gmp/clip v0.0.0-20191001134149-1458ba6a7cf5
-	git.wow.st/gmp/rand v0.0.0-20191001220155-a81bebfaf8b0
-	github.com/fsnotify/fsnotify v1.4.7
-	github.com/jcmdev0/gpgagent v0.0.0-20180509014935-5601b32d936c
-	golang.org/x/crypto v0.0.0-20190911031432-227b76d455e7
-	golang.org/x/image v0.0.0-20190910094157-69e4b8554b2a
-	gopkg.in/yaml.v2 v2.2.2
-)

go.sum

@@ -1,29 +0,0 @@
-gioui.org/ui v0.0.0-20190918172808-816f0e901fc6 h1:LvHEYxyOW7g+PhOiAm8Delc3AUv9EH219oDvaUMeKBw=
-gioui.org/ui v0.0.0-20190918172808-816f0e901fc6/go.mod h1:PssKPKlqVIeyaed+0w492Xc2NgX5M3n6oZKOAj5rxoE=
-git.wow.st/gmp/clip v0.0.0-20191001134149-1458ba6a7cf5 h1:OKeTjZST+/TKvtdA258NXJH+/gIx/xwyZxKrAezNFvk=
-git.wow.st/gmp/clip v0.0.0-20191001134149-1458ba6a7cf5/go.mod h1:NLdpaBoMQNFqncwP8OVRNWUDw1Kt9XWm3snfT7cXu24=
-git.wow.st/gmp/rand v0.0.0-20191001220006-66bfa936ad6b h1:yu3SnygEUgkNJ3xBXLkWHws7tki5qveZ6Kk7ybWj1GQ=
-git.wow.st/gmp/rand v0.0.0-20191001220006-66bfa936ad6b/go.mod h1:8+2Gwnrpc5yuk8Wp6cBhxvGcNLumYiPbQ7n0SQ8h29A=
-git.wow.st/gmp/rand v0.0.0-20191001220155-a81bebfaf8b0 h1:08wP00wvbDINsct1fzKV1xGGLvvtNsSb2X4CtIdpBzM=
-git.wow.st/gmp/rand v0.0.0-20191001220155-a81bebfaf8b0/go.mod h1:8+2Gwnrpc5yuk8Wp6cBhxvGcNLumYiPbQ7n0SQ8h29A=
-github.com/fsnotify/fsnotify v1.4.7 h1:IXs+QLmnXW2CcXuY+8Mzv/fWEsPGWxqefPtCP5CnV9I=
-github.com/fsnotify/fsnotify v1.4.7/go.mod h1:jwhsz4b93w/PPRr/qN1Yymfu8t87LnFCMoQvtojpjFo=
-github.com/jcmdev0/gpgagent v0.0.0-20180509014935-5601b32d936c h1:DCnjNrPDSEslcqqBgcZBxlLUIhk2elQVyf2V+HkyxJI=
-github.com/jcmdev0/gpgagent v0.0.0-20180509014935-5601b32d936c/go.mod h1:vdJ2op9pzpbH8CbpYKYBD6zjURqDY13PmnVn2I/uYBs=
-golang.org/x/crypto v0.0.0-20190308221718-c2843e01d9a2/go.mod h1:djNgcEr1/C05ACkg1iLfiJU5Ep61QUkGW8qpdssI0+w=
-golang.org/x/crypto v0.0.0-20190911031432-227b76d455e7 h1:0hQKqeLdqlt5iIwVOBErRisrHJAN57yOiPRQItI20fU=
-golang.org/x/crypto v0.0.0-20190911031432-227b76d455e7/go.mod h1:yigFU9vqHzYiE8UmvKecakEJjdnWj3jj499lnFckfCI=
-golang.org/x/image v0.0.0-20190703141733-d6a02ce849c9/go.mod h1:FeLwcggjj3mMvU+oOTbSwawSJRM1uh48EjtB4UJZlP0=
-golang.org/x/image v0.0.0-20190910094157-69e4b8554b2a h1:gHevYm0pO4QUbwy8Dmdr01R5r1BuKtfYqRqF0h/Cbh0=
-golang.org/x/image v0.0.0-20190910094157-69e4b8554b2a/go.mod h1:FeLwcggjj3mMvU+oOTbSwawSJRM1uh48EjtB4UJZlP0=
-golang.org/x/net v0.0.0-20190404232315-eb5bcb51f2a3/go.mod h1:t9HGtf8HONx5eT2rtn7q6eTqICYqUVnKs3thJo3Qplg=
-golang.org/x/sys v0.0.0-20190215142949-d0b11bdaac8a/go.mod h1:STP8DvDyc/dI5b8T5hshtkjS+E42TnysNCUPdjciGhY=
-golang.org/x/sys v0.0.0-20190412213103-97732733099d/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs=
-golang.org/x/sys v0.0.0-20190813064441-fde4db37ae7a h1:aYOabOQFp6Vj6W1F80affTUvO9UxmJRx8K0gsfABByQ=
-golang.org/x/sys v0.0.0-20190813064441-fde4db37ae7a/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs=
-golang.org/x/text v0.3.0 h1:g61tztE5qeGQ89tm6NTjjM9VPIm088od1l6aSorWRWg=
-golang.org/x/text v0.3.0/go.mod h1:NqM8EUOU14njkJ3fqMW+pc6Ldnwhi/IjpwHt7yyuwOQ=
-gopkg.in/check.v1 v0.0.0-20161208181325-20d25e280405 h1:yhCVgyC4o1eVCa2tZl7eS0r+SDo693bJlVdllGtEeKM=
-gopkg.in/check.v1 v0.0.0-20161208181325-20d25e280405/go.mod h1:Co6ibVJAznAaIkqp8huTwlJQCZ016jof/cbN4VW5Yz0=
-gopkg.in/yaml.v2 v2.2.2 h1:ZCJp+EgiOT7lHqUV2J862kp8Qj64Jo6az82+3Td9dZw=
-gopkg.in/yaml.v2 v2.2.2/go.mod h1:hI93XBmqTisBFMUTm0b8Fm+jr3Dg1NNxqwp+5A1VGuI=

impl_android.go

@@ -1,24 +1,98 @@
 //+build android
+//go:generate mkdir -p classes
+//go:generate javac -bootclasspath $ANDROID_HOME/platforms/android-29/android.jar -classpath openpgp-api.jar -d classes PgpConnect.java
+//go:generate jar cf PgpConnect.jar -C classes .
+//go:generate rm -rf classes
 
 package passgo
 
+/*
+#import <jni.h>
+#import "jni_android.h"
+*/
+import "C"
+
 import (
 	"fmt"
+	"io/ioutil"
+	"path"
+	"log"
+	"strings"
+	"sync"
+
+	"gioui.org/app"
+	_ "gioui.org/app/permission/storage"
 )
 
+var (
+	h *app.Handle
+	waitch chan struct{}
+	w *app.Window
+	pgp PGP
+	InitPgpOnce sync.Once
+)
+
+func init() {
+	waitch = make(chan struct{})
+}
+
+//export installComplete
+func installComplete(env *C.JNIEnv, class C.jclass, p C.jobject) {
+	log.Printf("InstallComplete()")
+	pgp = PGP(C.NewGlobalRef(env, p))
+	close(waitch)
+}
+
+func InitPgp(x *app.Window) {
+	InitPgpOnce.Do(func() {
+		log.Printf("InitPgp()")
+		w = x
+		h = app.PlatformHandle()
+		SetJVM(h.JVM, h.Context)
+		RunInJVM(func(env *JNIEnv) {
+			C.InitPgpConnect(env)
+		})
+		w.RegisterDelegate("st/wow/git/passgo/PgpConnect")
+	})
+}
+
+func stopPgp() {
+	waitch = make(chan struct{})
+	w = nil
+}
+
+func connect() {
+	<-waitch
+}
+
 //Clip copies a string to the clipboard
 func Clip(x string) {
-	fmt.Println("Clipboard not implemented for this platform")
+	pgp.Clip(x)
 }
 
 func (s *Store) nativeDecrypt(name string) (string, error) {
-	return "", fmt.Errorf("NOT IMPLEMENTED")
+	connect()
+	data, err := ioutil.ReadFile(path.Join(s.Dir, strings.Join([]string{name, ".gpg"}, "")))
+	if err != nil {
+		return "", fmt.Errorf("Error reading file: %s", err)
+	}
+	ret, err := pgp.Decrypt(string(data))
+	return ret, err
 }
 
 func (s *Store) nativeEncrypt(pw string) ([]byte, error) {
-	return nil, fmt.Errorf("NOT IMPLEMENTED")
+	connect()
+	ret, err := pgp.Encrypt(s.Id, pw)
+	return []byte(ret), err
 }
 
 func nativeIdentities() ([]string, error) {
-	return nil, fmt.Errorf("NOT IMPLEMENTED")
+	//NOT IMPLEMENTED ON ANDROID
+	return []string{}, nil
+	log.Printf("nativeIdentities()")
+	connect()
+	log.Printf("calling pgp.GetId()")
+	id, err := pgp.GetId()
+	log.Printf("pgp.GetId() returned")
+	return []string{id}, err
 }

jni_android.c

@@ -0,0 +1,146 @@
+
+#include <stdlib.h>
+#include <stdio.h>
+#include <jni.h>
+#include <android/log.h>
+#include "jni_android.h"
+#include "_cgo_export.h"
+
+static jobject gClassLoader;
+static jmethodID gFindClassMethod;
+
+void
+SetLoader(JNIEnv* env, jobject context) {
+	jclass cclass = (*env)->GetObjectClass(env, context);
+	jmethodID gcl_id = (*env)->GetMethodID(env, cclass, "getClassLoader", "()Ljava/lang/ClassLoader;");
+	jobject loader = (*env)->CallObjectMethod(env, context, gcl_id);
+	gClassLoader = (*env)->NewGlobalRef(env, loader);
+	jclass lclass = (*env)->GetObjectClass(env, loader);
+	gFindClassMethod = (*env)->GetMethodID(env, lclass, "findClass", "(Ljava/lang/String;)Ljava/lang/Class;");
+}
+
+jclass
+FindClass(JNIEnv* env, char* name) {
+	jstring strClassName = (*env)->NewStringUTF(env, name);
+	return (*env)->CallObjectMethod(env, gClassLoader, gFindClassMethod, strClassName);
+}
+
+jobject
+CreateObject(JNIEnv* env, jclass cls) {
+	jmethodID init = (*env)->GetMethodID(env, cls, "<init>", "()V");
+	return (*env)->NewObject(env, cls, init);
+}
+
+void
+InitPgpConnect(JNIEnv* env) {
+	jclass cls = FindClass(env, "st/wow/git/passgo/PgpConnect");
+	printf("class = %p", cls);
+
+	static const JNINativeMethod actMethods[] = {
+		{
+			.name = "activityResult",
+			.signature = "(II)V",
+			.fnPtr = activityResult
+		},
+		{
+			.name = "stringResult",
+			.signature = "(ILjava/lang/String;)V",
+			.fnPtr = stringResult
+		},
+		{
+			.name = "installComplete",
+			.signature = "(Lst/wow/git/passgo/PgpConnect;)V",
+			.fnPtr = installComplete
+		},
+	};
+
+	(*env)->RegisterNatives(env, cls, actMethods, sizeof(actMethods)/sizeof(actMethods[0]));
+}
+
+void
+GetId(JNIEnv* env, jobject p, int chint) {
+	jclass cls = (*env)->GetObjectClass(env, p);
+	printf("GetId(): cls = %p", cls);
+	jmethodID mid = (*env)->GetMethodID(env, cls, "GetId", "(I)V");
+	printf("GetId(): mid = %p", mid);
+	(*env)->CallObjectMethod(env, p, mid, chint);
+}
+
+void
+Decrypt(JNIEnv* env, jobject p, char* cdata, int datalen, int chint) {
+	jbyteArray data = (*env)->NewByteArray(env, datalen);
+	if (datalen > 0) {
+		(*env)->SetByteArrayRegion(env, data, 0, datalen, cdata);
+	}
+	jclass cls = (*env)->GetObjectClass(env, p);
+	jmethodID mid = (*env)->GetMethodID(env, cls, "Decrypt", "([BI)V");
+	(*env)->CallVoidMethod(env, p, mid, data, chint);
+}
+
+void stringResult(JNIEnv* env, jclass cls, jint requestCode, jobject response) {
+	if (response == 0) {
+		goStringResult(env, cls, requestCode, 0);
+	} else {
+		char* str = (*env)->GetStringUTFChars(env, response, NULL);
+		goStringResult(env, cls, requestCode, str);
+	}
+}
+
+void
+Encrypt(JNIEnv* env, jobject p, char* cid, int idlen, char* cdata, int datalen, int chint) {
+	jbyteArray id = (*env)->NewByteArray(env, idlen);
+	if (idlen > 0) {
+		(*env)->SetByteArrayRegion(env, id, 0, idlen, cid);
+	}
+	jbyteArray data = (*env)->NewByteArray(env, datalen);
+	if (datalen > 0) {
+		(*env)->SetByteArrayRegion(env, data, 0, datalen, cdata);
+	}
+	jclass cls = (*env)->GetObjectClass(env, p);
+	jmethodID mid = (*env)->GetMethodID(env, cls, "Encrypt", "([B[BI)V");
+	(*env)->CallVoidMethod(env, p, mid, id, data, chint);
+}
+
+void
+Clip(JNIEnv* env, jobject p, char* cdata, int datalen) {
+	jbyteArray data = (*env)->NewByteArray(env, datalen);
+	if (datalen > 0) {
+		(*env)->SetByteArrayRegion(env, data, 0, datalen, cdata);
+	}
+	jclass cls = (*env)->GetObjectClass(env, p);
+	jmethodID mid = (*env)->GetMethodID(env, cls, "Clip", "([B)V");
+	(*env)->CallVoidMethod(env, p, mid, data);
+}
+
+void CallVoidMethod(JNIEnv *env, jobject obj, jmethodID methodID) {
+	(*env)->CallVoidMethod(env, obj, methodID);
+}
+
+void CallVoidMethod1(JNIEnv *env, jobject obj, jmethodID methodID, jobject arg) {
+	(*env)->CallVoidMethod(env, obj, methodID, arg);
+}
+
+jint CallIntMethod(JNIEnv *env, jobject obj, jmethodID methodID) {
+	return (*env)->CallIntMethod(env, obj, methodID);
+}
+
+jmethodID GetMethodID(JNIEnv *env, jclass clazz, const char *name, const char *sig) {
+	return (*env)->GetMethodID(env, clazz, name, sig);
+}
+
+jint GetEnv(JavaVM *vm, JNIEnv **env, jint version) {
+	return (*vm)->GetEnv(vm, (void **)env, version);
+}
+
+jint AttachCurrentThread(JavaVM *vm, JNIEnv **p_env, void *thr_args) {
+	return (*vm)->AttachCurrentThread(vm, p_env, thr_args);
+}
+
+jint DetachCurrentThread(JavaVM *vm) {
+	return (*vm)->DetachCurrentThread(vm);
+}
+
+jobject
+NewGlobalRef(JNIEnv *env, jobject o) {
+	return (*env)->NewGlobalRef(env, o);
+}

jni_android.go

@@ -0,0 +1,195 @@
+package passgo
+
+/*
+#cgo LDFLAGS: -landroid -llog
+
+#include <stdlib.h>
+#include "jni_android.h"
+*/
+import "C"
+
+import (
+	"errors"
+	"fmt"
+	"log"
+	"runtime"
+	"unsafe"
+	"sync"
+)
+
+var theJVM *C.JavaVM
+type JNIEnv = C.JNIEnv
+
+func SetJVM(jvm, context uintptr) {
+	log.Print("set theJVM")
+	theJVM = (*C.JavaVM)(unsafe.Pointer(jvm))
+	RunInJVM(func(env *C.JNIEnv) {
+		C.SetLoader(env, (C.jobject)(context))
+	})
+}
+
+func FindClass(env *C.JNIEnv, name string) C.jclass {
+	cname := C.CString(name)
+	defer C.free(unsafe.Pointer(cname))
+	return C.FindClass((*C.JNIEnv)(env), cname)
+}
+
+func JniCallVoidMethod(env *C.JNIEnv, obj C.jobject, methodID C.jmethodID) {
+	C.CallVoidMethod(env, obj, methodID)
+}
+
+func JniCallVoidMethod1(env *C.JNIEnv, obj C.jobject, methodID C.jmethodID, arg uintptr) {
+	C.CallVoidMethod1(env, obj, methodID, C.jobject(unsafe.Pointer(arg)))
+}
+
+func JniCallIntMethod(env *C.JNIEnv, obj C.jobject, methodID C.jmethodID) int {
+	return (int)(C.CallIntMethod(env, obj, methodID))
+}
+
+func JniGetMethodID(env *C.JNIEnv, cls C.jclass, name, sig string) C.jmethodID {
+	cname := C.CString(name)
+	defer C.free(unsafe.Pointer(cname))
+	csig := C.CString(sig)
+	defer C.free(unsafe.Pointer(csig))
+	return C.GetMethodID(env, cls, cname, csig)
+}
+
+func CreateObject(env *C.JNIEnv, cls C.jclass) C.jobject {
+	return C.CreateObject(env, (C.jclass)(cls))
+}
+
+type PGP C.jobject
+
+func InitPgpConnect(env uintptr) {
+	C.InitPgpConnect((*C.JNIEnv)(unsafe.Pointer(env)))
+}
+
+//func NewPgpConnect(env uintptr, act uintptr) PGP {
+//	return (PGP)(C.NewPgpConnect((*C.JNIEnv)(unsafe.Pointer(env)), (C.jobject)(unsafe.Pointer(act))))
+//}
+
+var (
+	fragChans map[int]chan string
+	chmux sync.Mutex
+	chint int
+)
+
+func (p PGP) GetId() (string, error) {
+	ch := make(chan string)
+	chmux.Lock()
+	if fragChans == nil {
+		fragChans = make(map[int]chan string)
+	}
+	fragChans[chint] = ch
+	curint := chint
+	chint++
+	chmux.Unlock()
+
+	RunInJVM(func(env *JNIEnv) {
+		C.GetId(env, (C.jobject)(p), C.int(curint))
+	})
+	log.Printf("GetId(): waiting for response")
+	switch x := <-ch; {
+	case x == "":
+		log.Printf("GetId(): failed")
+		return "", fmt.Errorf("GetId failed")
+	default:
+		log.Printf("GetId(): got %s", x)
+		return x, nil
+	}
+}
+
+func (p PGP) Decrypt(data string) (string, error) {
+	ch := make(chan string)
+	chmux.Lock()
+	if fragChans == nil {
+		fragChans = make(map[int]chan string)
+	}
+	fragChans[chint] = ch
+	curint := chint
+	chint++
+	chmux.Unlock()
+
+	cdata := C.CString(data)
+	defer C.free(unsafe.Pointer(cdata))
+	RunInJVM(func(env *JNIEnv) {
+		C.Decrypt(env, (C.jobject)(p), (*C.char)(unsafe.Pointer(cdata)), (C.int)(len(data)), C.int(curint))
+	})
+	switch x := <-ch; {
+	case x == "":
+		return "", fmt.Errorf("Decryption failed")
+	default:
+		return x, nil
+	}
+}
+
+func (p PGP) Encrypt(id, data string) (string, error) {
+	ch := make(chan string)
+	chmux.Lock()
+	if fragChans == nil {
+		fragChans = make(map[int]chan string)
+	}
+	fragChans[chint] = ch
+	curint := chint
+	chint++
+	chmux.Unlock()
+
+	cid := C.CString(id)
+	defer C.free(unsafe.Pointer(cid))
+	cdata := C.CString(data)
+	defer C.free(unsafe.Pointer(cdata))
+	RunInJVM(func(env *JNIEnv) {
+		C.Encrypt(env, (C.jobject)(p), (*C.char)(unsafe.Pointer(cid)), (C.int)(len(id)), (*C.char)(unsafe.Pointer(cdata)), (C.int)(len(data)), C.int(curint))
+	})
+	switch x := <-ch; {
+	case x == "":
+		return "", fmt.Errorf("Encryption failed")
+	default:
+		return x, nil
+	}
+}
+
+func (p PGP) Clip(data string) {
+	cdata := C.CString(data)
+	defer C.free(unsafe.Pointer(cdata))
+	RunInJVM(func(env *JNIEnv) {
+		C.Clip(env, (C.jobject)(p), (*C.char)(unsafe.Pointer(cdata)), (C.int)(len(data)))
+	})
+}
+
+func RunInJVM(f func(env *C.JNIEnv)) {
+	runtime.LockOSThread()
+	defer runtime.UnlockOSThread()
+	var env *C.JNIEnv
+	var detach bool
+	if res := C.GetEnv(theJVM, &env, C.JNI_VERSION_1_6); res != C.JNI_OK {
+		if res != C.JNI_EDETACHED {
+			panic(fmt.Errorf("JNI GetEnv failed with error %d", res))
+		}
+		if C.AttachCurrentThread(theJVM, &env, nil) != C.JNI_OK {
+			panic(errors.New("runInJVM: AttachCurrentThread failed"))
+		}
+		detach = true
+	}
+
+	if detach {
+		defer func() {
+			C.DetachCurrentThread(theJVM)
+		}()
+	}
+	f(env)
+}
+
+//export activityResult
+func activityResult(env *C.JNIEnv, class C.jclass, request, result C.jint) {
+	log.Printf("activityResult (%d): %d", request, result)
+}
+
+//export goStringResult
+func goStringResult(env *C.JNIEnv, class C.jclass, request C.jint, str *C.char) {
+	fragChans[int(request)] <- C.GoString(str)
+	C.free(unsafe.Pointer(str))
+	chmux.Lock()
+	delete(fragChans, int(request))
+	chmux.Unlock()
+}

jni_android.h

@@ -0,0 +1,19 @@
+#include <jni.h>
+
+void SetLoader(JNIEnv* env, jobject context);
+jclass FindClass(JNIEnv* env, char* name);
+jobject CreateObject(JNIEnv* env, jclass cls);
+void InitPgpConnect(JNIEnv* env);
+void GetId(JNIEnv* env, jobject p, int chint);
+void Decrypt(JNIEnv* env, jobject p, char* cdata, int datalen, int chint);
+void Encrypt(JNIEnv* env, jobject p, char* cid, int idlen, char* cdata, int datalen, int chint);
+void Clip(JNIEnv* env, jobject p, char* cdata, int datalen);
+void stringResult(JNIEnv* env, jclass cls, jint requestCode, jobject response);
+void CallVoidMethod(JNIEnv *env, jobject obj, jmethodID methodID);
+void CallVoidMethod1(JNIEnv *env, jobject obj, jmethodID methodID, jobject arg);
+jint CallIntMethod(JNIEnv *env, jobject obj, jmethodID methodID);
+jmethodID GetMethodID(JNIEnv *env, jclass clazz, const char *name, const char *sig);
+jint GetEnv(JavaVM *vm, JNIEnv **env, jint version);
+jint AttachCurrentThread(JavaVM *vm, JNIEnv **p_env, void *thr_args);
+jint DetachCurrentThread(JavaVM *vm);
+jobject NewGlobalRef(JNIEnv *env, jobject o);

main.go

@@ -5,6 +5,7 @@ import (
 	"fmt"
 	"io"
 	"io/ioutil"
+	"log"
 	"os"
 	"os/user"
 	"path"
@@ -53,10 +54,23 @@ func GetStore(store *Store) error {
 	if err != nil {
 		return fmt.Errorf("Can't read .gpg-id: %s", err)
 	}
+	// extract only the email address portion of the ID (required
+	// for Android)
 	if id[len(id)-1] == '\n' {
 		id = id[:len(id)-1]
 	}
+	for i := len(id) - 1; i > 0; i-- {
+		if id[i] == '>' {
+			for j := i-1; j > 0; j-- {
+				if id[j] == '<' {
+					id = id[j+1:i]
+				}
+			}
+			break
+		}
+	}
 	store.Id = string(id)
+	log.Printf("ID = %s", store.Id)
 	return getKeyring()
 }
 
@@ -134,6 +148,8 @@ func (s *Store) list(level int, p string) ([]Pass, error) {
 	fd, err := os.Open(dir)
 	defer fd.Close()
 	if err != nil {
+		log.Printf("password store directory = %s", dir)
+		log.Printf("error is %s", err)
 		return nil, fmt.Errorf("Cannot open password store")
 	}
 	files, err := fd.Readdir(0)
@@ -262,6 +278,7 @@ func (s *Store) Decrypt(name string, prompts ...func() []byte) (string, error) {
 }
 
 func Identities() ([]string, error) {
+	log.Printf("Identities()")
 	getKeyring()
 	if useNative {
 		return nativeIdentities()