#2174 ✓fixreleased
Zvi Biener

Crash when using keybinding to move to another mailbox

Reported by Zvi Biener | November 12th, 2018 @ 04:45 PM

Crashes when using a keybinding to move to the next mailbox with a Displayed Count indicator.

I have a pretty tweaked keybinding file. I'm sure there are better ways to define the keys n, m, ",", and ".", but they've worked in the past. They issue arises when I press "n". Latest version of MM used.

The keybinding file and a crash report are attached. Happy to also send mailboxes.plist.

Comments and changes to this ticket

  • benny

    benny November 19th, 2018 @ 02:33 PM

    Do you still have this issue with the latest test release? I don't seem to be able to reproduce it.

    If you can, does it happen every time you hit n or only some times?

    (I have seen somewhat similar crashes reported, but I've not been able to reproduce any of them.)

  • Zvi Biener

    Zvi Biener November 19th, 2018 @ 03:43 PM

    I can no longer reproduce it. It was reproducible for about 8-10 hours (and every single time I hit 'n'), but then it stopped. No idea what the stopping correlated with. It was not because I updated MM.

    But now gone to MM 5566, and will update if I see anything.

  • benny

    benny November 20th, 2018 @ 01:42 PM

    • State changed from “new” to “cantreproduce”

    Thanks for the update.

  • Zvi Biener

    Zvi Biener November 26th, 2018 @ 02:30 PM

    This has started happening again. I've sent the crash reporting. Any other digging I can do?

  • benny

    benny November 26th, 2018 @ 02:34 PM

    Does it happen if you reduce it to selectNextCountedMailbox:?

    Does it matter which mailbox is selected and what the next mailbox is?

    (Looking for triggers since this would be much easier to fix if I can reproduce it.)

  • Zvi Biener

    Zvi Biener November 26th, 2018 @ 02:49 PM

    1. I think that when I first played with it, reducing to selectNextcountedMailbox did not make a difference. But don't act on that, I can't be 100% sure.

    2. It definitely did not matter which mailbox was selected and which was the next one.

    3. Although this happened several times in a row, I just continued using, and now I can no longer reproduce. Whatever this is, it is very very specific. I think now I'm aware enough about how specific at is that next time I can try to pin it down more... (at least I'm removing possible situations..) Sorry.

  • Zvi Biener

    Zvi Biener November 26th, 2018 @ 06:03 PM

    OK. I reloaded MailMate from backup, and can reproduce.

    The fault is with: "setFilterKey:", "unread".

    I've included an image, to explain. The crash happen when setFilterKey, unread (with no selectNextCountedMailbox) is used on the box "Mailing Lists." This is an empty box, which simply contains smart mailboxes. [I don't know why I did it this way, instead of another smart mailbox, but anyway].

    In fact, I'm not sure how I got it to show up with a folder icon, not the smart mailbox icon. In fact, it seems to be a smart mailbox, defined like this:

    { countKey = "none"; filter = "(#flags.flag = 'ListServs' and #flags.flag !=[x] '\Seen')"; name = "Mailing Lists"; parentUUID = "DCE5A74E-D001-4561-B3CA-1DA311C5EE5F"; uuid = "A6060EA5-56AC-46EB-A272-4CDFA46CB44E"; },

    Here are some relevant conditions:

    Crash happens 1) whether or not there are unread messages in the smart sub-mailboxes. 2) whether or not there is a displayed count on the mailbox. 3) the crash does NOT happen for the parent "Mailboxes" mailbox. 4) The crash still happen even when I remove all smart-sub-mailboxes. 5) still happen when I move the box from it's current location to another (even if I quite first, re-open and then try).

    I can overcome this problem very easily, now that I know what it is. I can simply switch the parent mailbox. Perhaps not a problem really worth pursuing, but FYI, that's what I found. If you'd like to me to check anything else, I can. But no worries if not.

  • Zvi Biener
  • Zvi Biener

    Zvi Biener November 26th, 2018 @ 06:07 PM

    Of course, I should add that this behavior disappeared for some reason, which was not a redefinition of mailboxes on my part. I'll keep the box around just to see if I can spot when the problem disappears.

  • benny

    benny November 28th, 2018 @ 02:01 PM

    • State changed from “cantreproduce” to “reproduced”

    At least right now I seem to be able to reproduce it using your Mailing Lists example. I'll look into it.

  • benny

    benny November 28th, 2018 @ 02:02 PM

    Hmm, I'm not getting the same crash type as you originally added to this ticket, but I am getting a crash related to setFilterKey:. Just to make clear that there just might be 2 issues.

  • benny

    benny November 28th, 2018 @ 02:17 PM

    • State changed from “reproduced” to “fixcommitted”
  • benny

    benny December 13th, 2018 @ 10:59 AM

    • State changed from “fixcommitted” to “fixreleased”

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.

New-ticket Create new ticket

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป

Mac OS X email client.

Shared Ticket Bins

People watching this ticket

Attachments

Pages