>>>>> In >>>>> Ted Zlatanov wrote: > Lines of code are not the only measure of code cleanliness; in this > specific case I found your logic hard to read while debugging an > unrelated problem so I tried to clean it up. You appear to be shortsighted. epg-context-* and epg-context-set-* are close to simple getter/setter functions of defstruct in CL (see the definition of epg-context-*). It looks even unnatural for me to use a temporary variable just to refer a slot value. Anyway, I'm going to commit the following change, which ensures the PASSPHRASE-CALLBACK slot have a cons-cell, and reduces the call to `epg-context-passphrase-callback' to 3 times.