Angband.oook.cz
Angband.oook.cz
AboutVariantsLadderForumCompetitionComicScreenshotsFunniesLinks

Go Back   Angband Forums > Angband > Development

Reply
 
Thread Tools Display Modes
Old October 10, 2011, 12:32   #1
zaimoni
Knight
 
zaimoni's Avatar
 
Join Date: Apr 2007
Posts: 590
zaimoni is on a distinguished road
flow.h|c/Zaiband stabilized

Well, that was embarrassing. Cf. Mercurial for the working version.

Code:
#define WRAPQUEUE_LEN 4*254+1

....

trailing_edge = (trailing_edge+1)%WRAPQUEUE_LEN;
The macro is not compliant with the intended coding standard here; WRAPQUEUE_LEN is supposed to be a primary expression i.e. wrapped in parentheses. In this case, the actual effect is ugly: instead of doing a modulo 4*254+1 increment, we update to an index with uinitialized data. Oops.
__________________
Zaiband: end the "I shouldn't have survived that" experience. V3.0.6 fork on Hg.
Zaiband 3.0.10 ETA Mar. 7 2011 (Yes, schedule slipped. Latest testing indicates not enough assert() calls to allow release.)
Z.C++: pre-alpha C/C++ compiler system (usable preprocessor). Also on Hg. Z.C++ 0.0.10 ETA December 31 2011
zaimoni is offline   Reply With Quote
Reply


Currently Active Users Viewing This Thread: 1 (0 members and 1 guests)
 
Thread Tools
Display Modes

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

BB code is On
Smilies are On
[IMG] code is On
HTML code is Off

Forum Jump

Similar Threads
Thread Thread Starter Forum Replies Last Post
Zaiband 3.0.9 alpha released zaimoni Variants 0 July 3, 2008 06:52
Zaiband 3.0.8 alpha released zaimoni Variants 0 March 30, 2008 09:45
Zaiband: Sidebar documentation zaimoni Variants 8 March 4, 2008 07:40
Zaiband 3.0.7 zaimoni Variants 4 November 6, 2007 09:32


All times are GMT +1. The time now is 23:04.


Powered by vBulletin® Version 3.8.11
Copyright ©2000 - 2019, vBulletin Solutions Inc.