From 4a35c9a728df05a99eff3c335ee7ec2ddf201cff Mon Sep 17 00:00:00 2001
From: Mark Wooding <mdw@distorted.org.uk>
Date: Sun, 24 May 2015 16:38:11 +0100
Subject: [PATCH] catacomb/pwsafe.py, pwsafe: Replace `PW''s MODE parameter
 with WRITEP flag.

This abstracts away from the GDBM interface slightly, and makes it a bit
more convenient for other implementations.
---
 catacomb/pwsafe.py | 8 ++++----
 pwsafe             | 6 +++---
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/catacomb/pwsafe.py b/catacomb/pwsafe.py
index 65aec71..1e8e4db 100644
--- a/catacomb/pwsafe.py
+++ b/catacomb/pwsafe.py
@@ -161,17 +161,17 @@ class PW (object):
   using the stored keys.
   """
 
-  def __init__(me, file, mode = 'r'):
+  def __init__(me, file, writep = False):
     """
     Initialize a PW object from the GDBM database in FILE.
 
-    MODE can be `r' for read-only access to the underlying database, or `w'
-    for read-write access.  Requests the database password from the Pixie,
+    If WRITEP is true, then allow write-access to the database; otherwise
+    allow read access only.  Requests the database password from the Pixie,
     which may cause interaction.
     """
 
     ## Open the database.
-    me.db = _G.open(file, mode)
+    me.db = _G.open(file, writep and 'w' or 'r')
 
     ## Find out what crypto to use.
     c = _C.gcciphers[me.db['cipher']]
diff --git a/pwsafe b/pwsafe
index a3365e7..d4a4547 100644
--- a/pwsafe
+++ b/pwsafe
@@ -97,7 +97,7 @@ def cmd_create(av):
 def cmd_changepp(av):
   if len(av) != 0:
     return 1
-  pw = PW(file, 'w')
+  pw = PW(file, writep = True)
   pw.changepp()
 
 def cmd_find(av):
@@ -122,7 +122,7 @@ def cmd_store(av):
     pp = stdin.readline()
   else:
     pp = av[1]
-  pw = PW(file, 'w')
+  pw = PW(file, writep = True)
   pw[av[0]] = chomp(pp)
 
 def cmd_copy(av):
@@ -169,7 +169,7 @@ def cmd_topixie(av):
 def cmd_del(av):
   if len(av) != 1:
     return 1
-  pw = PW(file, 'w')
+  pw = PW(file, writep = True)
   tag = av[0]
   try:
     del pw[tag]
-- 
2.11.0