<div dir="ltr"><div>I will keep updating my code in this pull-request:</div><div><br></div><a href="https://github.com/zhangkun83/password-store/pull/1">https://github.com/zhangkun83/password-store/pull/1</a><br><div><br></div><div class="gmail_extra"><div class="gmail_quote">On Mon, Aug 15, 2016 at 9:37 AM, mitfree <span dir="ltr"><<a href="mailto:lists@ampling.com" target="_blank">lists@ampling.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
I use passmenu to solve this same problem, but I'm willing to try and offer<br>
some constructive criticism.<br>
<span class=""><br>
> My change is on github (<a href="https://github.com/zhangkun83/password-store/commit/" rel="noreferrer" target="_blank">https://github.com/<wbr>zhangkun83/password-store/<wbr>commit/</a><br>
> a0ccc8f0508ff14169b1923e0273a5<wbr>7b8f2597ca).  It works great for me, and I would<br>
> like to merge this feature upstream.  Please let me know if I need to make<br>
> additional changes (e.g., tests and man pages), or implement this in a<br>
> different way. <br>
<br>
</span>## Safe Data Manipulation<br>
line 324: rm -f "$matches_file" "$matches_file_swap"<br>
Could you use a bash array instead of writing to disk? If the hidden<br>
file names don't need to leave the running process, I say try to keep it<br>
within bash (arrays and variables).<br>
<br>
arrayName=(Value1 Value2 Value3 .. ValueN)<br>
${arrName[index]}<br>
${#arrayName[@]}<br></blockquote><div><br></div><div>I refrained from using variables for the intermediate results because I was concerned about argument list expanded beyond the command line limit.  Interestingly, it seems internal commands like "echo" has special treatment and is not subject to such limit :) Arrays isn't more useful than plain variables, so I use variables instead.</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
## Regular Expression<br>
line 338: path=$(echo "$matches" | perl -pe 's/\e\[?.*?[\@-~]//g')<br>
Why not use sed or builtin?<br>
Who wants to install all of perl to run bash code?<br></blockquote><div><br></div><div>I googled around but didn't find other code that successfully remove ANSI color codes for me.</div><div>So I landed with the first solution that worked. I can look harder though.</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I hope you find this useful.<br>
<span class="HOEnZb"><font color="#888888"><br>
<br>
--<br>
At your service,<br>
mitfree<br>
<a href="https://ampling.com" rel="noreferrer" target="_blank">https://ampling.com</a><br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">Best wishes<br>Kun Zhang</div></div>
</div></div>