search

raymontag / keepassc

KeePassC is a curses-based password manager compatible to KeePass v.1.x and KeePassX
http://raymontag.github.com/keepassc/
ISC License
311 stars 23 forks source link

Couple of bugfixes for you... #32

Closed firecat53 closed 11 years ago

firecat53 commented 11 years ago

I think I fixed Ctrl-C handling in the editor by switching to raw() mode. Please test and make sure this doesn't break anything else....namely the Unicode handling! Question for you....when someone hits Ctrl-C from the editor or the main program, should they be prompted to save a changed database to avoid db corruption? I think you used to have it that way, but now it just stops.

Also fixed a small bug in dbbrowser (it was using the wrong inittext when editing the entry titles) and a couple of small bugs in the editor.

Thanks! Scott

raymontag commented 11 years ago

Hmm, I remember that I'd a problem with raw-mode as it result in non-handling of any flow control like Ctrl+D. I'll clone this and try it out.

firecat53 commented 11 years ago

Ctrl-C and Ctrl-Z work normally now with these commits. Ctrl-D doesn't do anything from the editor window, and from keepassc it seems to act the same as ^C....what are you expecting it to do in this context? Should it save the current editor buffer and quit the editor, or should it act the same as ^C? Sending the EOF signal (I believe that's what ^D does) doesn't seem to make sense in the keepassc context. I think as long as ^Z, ^C and ^\ work (those are the typical unix flow-control commands), then we should be ok, right?

raymontag commented 11 years ago

OK, I've checked the changes and everything works fine!