Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

Crash when using keybinding to move to another mailbox

#2174

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.

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

State: new
Milestone: none
Assigned to: benny benny

Activity

  1. benny
    benny

    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.)

    November 19th, 2018 @ 02:33 PM

  2. Zvi Biener
    Zvi Biener

    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.

    November 19th, 2018 @ 03:43 PM

  3. benny
    benny
    • State changed from new to cantreproduce

    Thanks for the update.

    November 20th, 2018 @ 01:42 PM

  4. Zvi Biener
    Zvi Biener

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

    November 26th, 2018 @ 02:30 PM

  5. benny
    benny

    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.)

    November 26th, 2018 @ 02:34 PM

  6. Zvi Biener
    Zvi Biener
    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.

    November 26th, 2018 @ 02:49 PM

  7. Zvi Biener
    Zvi Biener

    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.

    November 26th, 2018 @ 06:03 PM

  8. Zvi Biener
  9. Zvi Biener
    Zvi Biener

    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.

    November 26th, 2018 @ 06:07 PM

  10. benny
    benny
    • 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.

    November 28th, 2018 @ 02:01 PM

  11. benny
    benny

    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.

    November 28th, 2018 @ 02:02 PM

  12. benny
    benny
    • State changed from reproduced to fixcommitted

    November 28th, 2018 @ 02:17 PM

  13. benny
    benny
    • State changed from fixcommitted to fixreleased

    December 13th, 2018 @ 10:59 AM

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.