Archives/OpenTTD-patches
2
0
Fork 0
mirror of https://github.com/JGRennison/OpenTTD-patches.git synced 2024-11-17 21:25:40 +00:00
Code Issues Packages Projects Releases Wiki Activity

(svn r7388) -Fix (r7384 / r7368 / r3757): When adding a new news item for the first time (_latest_news

Browse Source 
is INVALID_NEWS), make it the value of _oldest_news, not zero as DeleteVehicleNews can
 change _oldest_news. Also remove the (i == INVALID_NEWS) return 0; guard from increaseIndex
 as it's only used in 2 places, nowhere else.
This commit is contained in:
Darkvater 2006-12-05 23:37:04 +00:00
parent 0e10e86137
commit f2a70e3e26
1 changed files with 6 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
news_gui.c
View File

@ -230,7 +230,7 @@ static void NewsWindowProc(Window *w, WindowEvent *e)
* queue and deals with overflows when increasing the index */ * queue and deals with overflows when increasing the index */
static inline NewsID increaseIndex(NewsID i) static inline NewsID increaseIndex(NewsID i)
{ {
if (i == INVALID_NEWS) return 0; assert(i != INVALID_NEWS);
return (i + 1) % MAX_NEWS; return (i + 1) % MAX_NEWS;
} }
@ -275,9 +275,11 @@ void AddNewsItem(StringID string, uint32 flags, uint data_a, uint data_b)
_forced_news = INVALID_NEWS; _forced_news = INVALID_NEWS;
if (_total_news < MAX_NEWS) _total_news++; if (_total_news < MAX_NEWS) _total_news++;
// make sure our pointer isn't overflowing /* Increase _latest_news. If we have no news yet, use _oldest news as an
* index. We cannot use 0 as _oldest_news can jump around due to
* DeleteVehicleNews */
l_news = _latest_news; l_news = _latest_news;
_latest_news = increaseIndex(_latest_news); _latest_news = (_latest_news == INVALID_NEWS) ? _oldest_news : increaseIndex(_latest_news);
/* If the fifo-buffer is full, overwrite the oldest entry */ /* If the fifo-buffer is full, overwrite the oldest entry */
if (l_news != INVALID_NEWS && _latest_news == _oldest_news) { if (l_news != INVALID_NEWS && _latest_news == _oldest_news) {
@ -489,7 +491,7 @@ static void MoveToNextItem(void)
if (_current_news != _latest_news) { if (_current_news != _latest_news) {
NewsItem *ni; NewsItem *ni;
_current_news = increaseIndex(_current_news); _current_news = (_current_news == INVALID_NEWS) ? _oldest_news : increaseIndex(_current_news);
ni = &_news_items[_current_news]; ni = &_news_items[_current_news];
// check the date, don't show too old items // check the date, don't show too old items