Skip to content

Commit

Permalink
Making a bunch of warnings and an error go away.
Browse files Browse the repository at this point in the history
Ruby 1.9.3 had a good point about "folderstyle = MBox" not actually
affecting anything on account of how I was setting a local variable
called that instead of the class member.

But really, that complaint about indentation being slightly off?  That's
excessively picky.
  • Loading branch information
dagbrown committed Jan 31, 2013
1 parent 2ed523e commit f2b0225
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 6 deletions.
2 changes: 1 addition & 1 deletion VERSION
@@ -1 +1 @@
1.10.5
1.10.6
4 changes: 2 additions & 2 deletions gurgitate-mail.RB
Expand Up @@ -361,7 +361,7 @@ module Gurgitate
rescue Exception
log "Error while executing #{configfilespec}: #{$!}"
$@.each { |tr| log "Backtrace: #{tr}" }
folderstyle = MBox
folderstyle MBox
save(spoolfile)
end
end
Expand All @@ -377,7 +377,7 @@ module Gurgitate
log "Error while executing rules: #{$!}"
$@.each { |tr| log "Backtrace: #{tr}" }
log "Attempting to save to spoolfile after error"
folderstyle = MBox
folderstyle MBox
save(spoolfile)
end
end
Expand Down
2 changes: 1 addition & 1 deletion gurgitate/deliver/mbox.RB
Expand Up @@ -41,7 +41,7 @@ module Gurgitate
f.print message
f.flock(File::LOCK_UN)
end
end
end
end
end
end
4 changes: 2 additions & 2 deletions gurgitate/deliver/mh.RB
Expand Up @@ -24,10 +24,10 @@ module Gurgitate
# I could put in a check for the path given in
# $HOME/.mh_profile, but Claws-Mail uses MH mailboxes and
# disregards $HOME/.mh_profile.
if File.stat(mailbox).directory? and
if( File.stat(mailbox).directory? and
not ( File.exists?(File.join(mailbox, "cur")) or
File.exists?(File.join(mailbox, "tmp")) or
File.exists?(File.join(mailbox, "new"))) then
File.exists?(File.join(mailbox, "new")) ) ) then
return MH
end
rescue Errno::ENOENT
Expand Down

0 comments on commit f2b0225

Please sign in to comment.