Daniel Brockman <[EMAIL PROTECTED]> writes:

>> The attached solution looks OK to me. Could it cause problems?
>
> I believe it could cause a slight problem since it assumes
> that `bongo-update-header-line-string' is only called in a
> few controlled situations.  This assumption seems bad.

Is it a bad assumption in practice? Will it ever happen in normal use?

Anyway, I'm looking at your code. I can't get it to work, maybe I'm
doing something wrong, but it /looks/ good!

I was thinking kind of in the same way. My idea was that it would
generally be useful to know whether a stop is between tracks. It could
be as simple as a variable that is bound non-nil in certain functions
(like `bongo-play-next'). I don't have an example of when this would
be useful, though, other than with the header line updates. What do
you think?

Back to your patch. I think that you may need to use
`with-deferred-bongo-status-indicator-updates' in more places, like
`bongo-replay-current'. 



_______________________________________________
bongo-devel mailing list
[email protected]
http://lists.nongnu.org/mailman/listinfo/bongo-devel

Reply via email to