Archive of RubyForge sup-talk mailing list
 help / color / mirror / Atom feed
* [sup-talk] Missing the save index option (patch attached)
@ 2010-01-17 12:36 Israel Herraiz
  2010-01-17 15:25 ` William Morgan
  2010-01-17 23:22 ` William Morgan
  0 siblings, 2 replies; 4+ messages in thread
From: Israel Herraiz @ 2010-01-17 12:36 UTC (permalink / raw)
  To: sup-talk

Hi all,

since the last changes to get rid of the save option and to migrate to
the Xapian index, I am really missing the save index option.

Currently, when Sup exits, all the changes to the index are flushed,
and in my case it takes ages because I am a heavy email load user. For
me that is a problem, because my Sup is always open, and I usually
suspend my laptop. From time to time, my laptop won't wake up, and I
am forced to do an unclean reboot. When I open again Sup, all the
changes I did (archiving threads, etc) are lost because the index was
not flushed.

I know that I can use the XAPIAN_FLUSH_THRESHOLD option, but if I use
that option, when the threshold is reached it interrupts me while I am
dealing with my email, and I have to wait some seconds until the index
is flushed. This behavior is very annoying.

I don't know if someone else in this list is also missing the
possibility of flushing the index from Sup. I have done the attached
changes to my Sup, that recovers a sort-of the old save index
option. I am sending to the list just in case the patch is useful for
others too. If there are not strong objections to this change, I
suggest to merge it with Sup.

Cheers,
Israel

---
 lib/sup/modes/thread-index-mode.rb |    7 +++++++
 1 files changed, 7 insertions(+), 0 deletions(-)

diff --git a/lib/sup/modes/thread-index-mode.rb b/lib/sup/modes/thread-index-mode.rb
index a5bd344..3a1bb76 100644
--- a/lib/sup/modes/thread-index-mode.rb
+++ b/lib/sup/modes/thread-index-mode.rb
@@ -37,6 +37,7 @@ EOS
     k.add :toggle_spam, "Mark/unmark thread as spam", 'S'
     k.add :toggle_deleted, "Delete/undelete thread", 'd'
     k.add :kill, "Kill thread (never to be seen in inbox again)", '&'
+    k.add :flush_index, "Flush all changes now", '$'
     k.add :jump_to_next_new, "Jump to next new thread", :tab
     k.add :reply, "Reply to latest message in a thread", 'r'
     k.add :reply_all, "Reply to all participants of the latest message in a thread", 'G'
@@ -449,6 +450,12 @@ EOS
     multi_kill [t]
   end
 
+  def flush_index
+    @flush_id = BufferManager.say "Flushing index..."
+    Index.save_index
+    BufferManager.clear @flush_id
+  end
+  
   ## m-m-m-m-MULTI-KILL
   def multi_kill threads
     UndoManager.register "killing #{threads.size.pluralize 'thread'}" do
-- 
1.6.6
_______________________________________________
sup-talk mailing list
sup-talk@rubyforge.org
http://rubyforge.org/mailman/listinfo/sup-talk


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [sup-talk] Missing the save index option (patch attached)
  2010-01-17 12:36 [sup-talk] Missing the save index option (patch attached) Israel Herraiz
@ 2010-01-17 15:25 ` William Morgan
  2010-01-17 16:12   ` Ben Walton
  2010-01-17 23:22 ` William Morgan
  1 sibling, 1 reply; 4+ messages in thread
From: William Morgan @ 2010-01-17 15:25 UTC (permalink / raw)
  To: sup-talk

Reformatted excerpts from Israel Herraiz's message of 2010-01-17:
> I don't know if someone else in this list is also missing the
> possibility of flushing the index from Sup. I have done the attached
> changes to my Sup, that recovers a sort-of the old save index option.
> I am sending to the list just in case the patch is useful for others
> too. If there are not strong objections to this change, I suggest to
> merge it with Sup.

I believe we've discussed this before and decided it would be a good
idea. Any objections to adding this to the insta-save branch?

(Also has the nice side-effect of making '$' not go to the bottom of the
screen, which was driving me batty.)
-- 
William <wmorgan-sup@masanjin.net>
_______________________________________________
sup-talk mailing list
sup-talk@rubyforge.org
http://rubyforge.org/mailman/listinfo/sup-talk


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [sup-talk] Missing the save index option (patch attached)
  2010-01-17 15:25 ` William Morgan
@ 2010-01-17 16:12   ` Ben Walton
  0 siblings, 0 replies; 4+ messages in thread
From: Ben Walton @ 2010-01-17 16:12 UTC (permalink / raw)
  To: sup-talk


[-- Attachment #1.1: Type: text/plain, Size: 440 bytes --]

Excerpts from William Morgan's message of Sun Jan 17 10:25:59 -0500 2010:

> (Also has the nice side-effect of making '$' not go to the bottom of the
> screen, which was driving me batty.)

Yes, that has been hard for me to unlearn too.

-Ben
-- 
Ben Walton
Systems Programmer - CHASS
University of Toronto
C:416.407.5610 | W:416.978.4302

GPG Key Id: 8E89F6D2; Key Server: pgp.mit.edu
Contact me to arrange for a CAcert assurance meeting.

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 189 bytes --]

[-- Attachment #2: Type: text/plain, Size: 140 bytes --]

_______________________________________________
sup-talk mailing list
sup-talk@rubyforge.org
http://rubyforge.org/mailman/listinfo/sup-talk

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [sup-talk] Missing the save index option (patch attached)
  2010-01-17 12:36 [sup-talk] Missing the save index option (patch attached) Israel Herraiz
  2010-01-17 15:25 ` William Morgan
@ 2010-01-17 23:22 ` William Morgan
  1 sibling, 0 replies; 4+ messages in thread
From: William Morgan @ 2010-01-17 23:22 UTC (permalink / raw)
  To: sup-talk

Reformatted excerpts from Israel Herraiz's message of 2010-01-17:
> I am sending to the list just in case the patch is useful for others
> too. If there are not strong objections to this change, I suggest to
> merge it with Sup.

Applied to insta-save and remerged into next. Thanks!
-- 
William <wmorgan-sup@masanjin.net>
_______________________________________________
sup-talk mailing list
sup-talk@rubyforge.org
http://rubyforge.org/mailman/listinfo/sup-talk


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2010-01-17 23:23 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-01-17 12:36 [sup-talk] Missing the save index option (patch attached) Israel Herraiz
2010-01-17 15:25 ` William Morgan
2010-01-17 16:12   ` Ben Walton
2010-01-17 23:22 ` William Morgan

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox