Re: Nibbles patches



I'm adding games-list to CC on this email, in the off chance someone
sees this and becomes interested in our invisible worm woes.

On Mon, 2014-08-18 at 11:11 -0400, Bryan Quigley wrote:
Hi Michael,

Just wanted to check if you wanted to commit anything else for Nibbles
before I run a release.

https://bugzilla.gnome.org/show_bug.cgi?id=733424
https://bugzilla.gnome.org/show_bug.cgi?id=733462
https://bugzilla.gnome.org/show_bug.cgi?id=654072

I can commit the first two if you want.  I'm not sure of the status of
the last one.. (there may be more that I missed...)

Thanks!
Bryan

I was indeed hoping to get to the Nibbles patches today, but I'm not
really sure if I will or not. Today is only really a deadline for the
"remove end game button" patch, which I definitely will try to land; the
rest of these we can commit after UI freeze because they are just
bugfixes.

I think I'm ready to commit all of the patches, but IIRC there were
still two problems:

* Sometimes when you respawn, the worm gets placed on the wrong squares,
but it's rare and I never figured out why.
* Sometimes when you go into the teleporter, our hacked up function to
determine the direction the worm is going in does not work, but it's
rare and I can't figure out why.

I think the game is still less buggy after my set of patches than
before, I'm just not certain whether we want to add the assertions or
not.  I think it'd be better to crash than let it break, since
triggering a distro bug reporting tool seems better than letting the
player die on an invisible worm, but it's entirely possible the user
would not notice the bug at all if we do not crash, and that might be
better than the bug reporting tool.

So I'm curious what your thoughts are on that. Especially if you have
time to help debug, that'd be great, but I know you're busy with other
things.

It might be best if you let me do today's release; that way we don't
have to worry about coordinating.

Cheers,

Michael

Attachment: signature.asc
Description: This is a digitally signed message part



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]