--- a/src/fileio.cpp Thu May 08 13:06:12 2008 +0000
+++ b/src/fileio.cpp Thu May 08 13:20:54 2008 +0000
@@ -27,7 +27,7 @@
struct Fio {
byte *buffer, *buffer_end; ///< position pointer in local buffer and last valid byte of buffer
- uint32 pos; ///< current (system) position in file
+ size_t pos; ///< current (system) position in file
FILE *cur_fh; ///< current file handle
const char *filename; ///< current filename
FILE *handles[MAX_FILE_SLOTS]; ///< array of file handles we can have open
@@ -125,7 +125,7 @@
return (FioReadWord() << 16) | b;
}
-void FioReadBlock(void *ptr, uint size)
+void FioReadBlock(void *ptr, size_t size)
{
FioSeekTo(FioGetPos(), SEEK_SET);
_fio.pos += fread(ptr, 1, size, _fio.cur_fh);
@@ -480,7 +480,7 @@
TarHeader th;
char buf[sizeof(th.name) + 1], *end;
char name[sizeof(th.prefix) + 1 + sizeof(th.name) + 1];
- int num = 0, pos = 0;
+ size_t num = 0, pos = 0;
/* Make a char of 512 empty bytes */
char empty[512];
@@ -499,7 +499,7 @@
}
name[0] = '\0';
- int len = 0;
+ size_t len = 0;
/* The prefix contains the directory-name */
if (th.prefix[0] != '\0') {
@@ -550,7 +550,7 @@
return true;
}
-static int ScanPathForTarFiles(const char *path, int basepath_length)
+static int ScanPathForTarFiles(const char *path, size_t basepath_length)
{
extern bool FiosIsValidFile(const char *path, const struct dirent *ent, struct stat *sb);
--- a/src/fileio.h Thu May 08 13:06:12 2008 +0000
+++ b/src/fileio.h Thu May 08 13:20:54 2008 +0000
@@ -18,7 +18,7 @@
uint32 FioReadDword();
void FioCloseAll();
void FioOpenFile(int slot, const char *filename);
-void FioReadBlock(void *ptr, uint size);
+void FioReadBlock(void *ptr, size_t size);
void FioSkipBytes(int n);
void FioCreateDirectory(const char *filename);
@@ -70,8 +70,8 @@
};
struct TarFileListEntry {
TarListEntry *tar;
- int size;
- int position;
+ size_t size;
+ size_t position;
};
typedef std::map<std::string, TarListEntry *> TarList;
typedef std::map<std::string, TarFileListEntry> TarFileList;
--- a/src/network/network_gui.cpp Thu May 08 13:06:12 2008 +0000
+++ b/src/network/network_gui.cpp Thu May 08 13:20:54 2008 +0000
@@ -1778,7 +1778,7 @@
{
static char _chat_tab_completion_buf[lengthof(_edit_str_net_buf)];
Textbuf *tb = &WP(w, chatquerystr_d).text;
- uint len, tb_len;
+ size_t len, tb_len;
uint item;
char *tb_buf, *pre_buf;
const char *cur_name;
@@ -1799,8 +1799,8 @@
/* We are pressing TAB again on the same name, is there an other name
* that starts with this? */
if (!second_scan) {
- uint offset;
- uint length;
+ size_t offset;
+ size_t length;
/* If we are completing at the begin of the line, skip the ': ' we added */
if (tb_buf == pre_buf) {
--- a/src/network/network_udp.cpp Thu May 08 13:06:12 2008 +0000
+++ b/src/network/network_udp.cpp Thu May 08 13:20:54 2008 +0000
@@ -205,7 +205,7 @@
const GRFConfig *in_reply[NETWORK_MAX_GRF_COUNT];
uint8 in_reply_count = 0;
- uint packet_len = 0;
+ size_t packet_len = 0;
DEBUG(net, 6, "[udp] newgrf data request from %s:%d", inet_ntoa(client_addr->sin_addr), ntohs(client_addr->sin_port));
--- a/src/newgrf.cpp Thu May 08 13:06:12 2008 +0000
+++ b/src/newgrf.cpp Thu May 08 13:20:54 2008 +0000
@@ -96,7 +96,7 @@
static GrfDataType _grf_data_type;
-typedef void (*SpecialSpriteHandler)(byte *buf, int len);
+typedef void (*SpecialSpriteHandler)(byte *buf, size_t len);
enum {
MAX_STATIONS = 256,
@@ -152,7 +152,7 @@
DEBUG(grf, severity, "[%s:%d] %s", _cur_grfconfig->filename, _nfo_line, buf);
}
-static inline bool check_length(int real, int wanted, const char *str)
+static inline bool check_length(size_t real, size_t wanted, const char *str)
{
if (real >= wanted) return true;
grfmsg(0, "%s: Invalid pseudo sprite length %d (expected %d)!", str, real, wanted);
@@ -2248,7 +2248,7 @@
}
/* Action 0x00 */
-static void FeatureChangeInfo(byte *buf, int len)
+static void FeatureChangeInfo(byte *buf, size_t len)
{
byte *bufend = buf + len;
@@ -2363,7 +2363,7 @@
}
/* Action 0x00 (GLS_SAFETYSCAN) */
-static void SafeChangeInfo(byte *buf, int len)
+static void SafeChangeInfo(byte *buf, size_t len)
{
if (!check_length(len, 6, "SafeChangeInfo")) return;
buf++;
@@ -2386,7 +2386,7 @@
}
/* Action 0x00 (GLS_RESERVE) */
-static void ReserveChangeInfo(byte *buf, int len)
+static void ReserveChangeInfo(byte *buf, size_t len)
{
byte *bufend = buf + len;
@@ -2482,7 +2482,7 @@
}
/* Action 0x01 */
-static void NewSpriteSet(byte *buf, int len)
+static void NewSpriteSet(byte *buf, size_t len)
{
/* <01> <feature> <num-sets> <num-ent>
*
@@ -2518,7 +2518,7 @@
}
/* Action 0x01 (SKIP) */
-static void SkipAct1(byte *buf, int len)
+static void SkipAct1(byte *buf, size_t len)
{
if (!check_length(len, 4, "SkipAct1")) return;
buf++;
@@ -2577,7 +2577,7 @@
}
/* Action 0x02 */
-static void NewSpriteGroup(byte *buf, int len)
+static void NewSpriteGroup(byte *buf, size_t len)
{
/* <02> <feature> <set-id> <type/num-entries> <feature-specific-data...>
*
@@ -3182,7 +3182,7 @@
/* Action 0x03 */
-static void FeatureMapSpriteGroup(byte *buf, int len)
+static void FeatureMapSpriteGroup(byte *buf, size_t len)
{
/* <03> <feature> <n-id> <ids>... <num-cid> [<cargo-type> <cid>]... <def-cid>
* id-list := [<id>] [id-list]
@@ -3267,7 +3267,7 @@
}
/* Action 0x04 */
-static void FeatureNewName(byte *buf, int len)
+static void FeatureNewName(byte *buf, size_t len)
{
/* <04> <veh-type> <language-id> <num-veh> <offset> <data...>
*
@@ -3421,7 +3421,7 @@
}
/* Action 0x05 */
-static void GraphicsNew(byte *buf, int len)
+static void GraphicsNew(byte *buf, size_t len)
{
/* <05> <graphics-type> <num-sprites> <other data...>
*
@@ -3535,7 +3535,7 @@
}
/* Action 0x05 (SKIP) */
-static void SkipAct5(byte *buf, int len)
+static void SkipAct5(byte *buf, size_t len)
{
if (!check_length(len, 2, "SkipAct5")) return;
buf++;
@@ -3711,7 +3711,7 @@
}
/* Action 0x06 */
-static void CfgApply(byte *buf, int len)
+static void CfgApply(byte *buf, size_t len)
{
/* <06> <param-num> <param-size> <offset> ... <FF>
*
@@ -3799,7 +3799,7 @@
/* Action 0x07 */
/* Action 0x09 */
-static void SkipIf(byte *buf, int len)
+static void SkipIf(byte *buf, size_t len)
{
/* <07/09> <param-num> <param-size> <condition-type> <value> <num-sprites>
*
@@ -3954,7 +3954,7 @@
/* Action 0x08 (GLS_FILESCAN) */
-static void ScanInfo(byte *buf, int len)
+static void ScanInfo(byte *buf, size_t len)
{
if (!check_length(len, 8, "Info")) return;
buf++;
@@ -3981,7 +3981,7 @@
}
/* Action 0x08 */
-static void GRFInfo(byte *buf, int len)
+static void GRFInfo(byte *buf, size_t len)
{
/* <08> <version> <grf-id> <name> <info>
*
@@ -4005,7 +4005,7 @@
}
/* Action 0x0A */
-static void SpriteReplace(byte *buf, int len)
+static void SpriteReplace(byte *buf, size_t len)
{
/* <0A> <num-sets> <set1> [<set2> ...]
* <set>: <num-sprites> <first-sprite>
@@ -4041,7 +4041,7 @@
}
/* Action 0x0A (SKIP) */
-static void SkipActA(byte *buf, int len)
+static void SkipActA(byte *buf, size_t len)
{
buf++;
uint8 num_sets = grf_load_byte(&buf);
@@ -4057,7 +4057,7 @@
}
/* Action 0x0B */
-static void GRFLoadError(byte *buf, int len)
+static void GRFLoadError(byte *buf, size_t len)
{
/* <0B> <severity> <language-id> <message-id> [<message...> 00] [<data...>] 00 [<parnum>]
*
@@ -4167,7 +4167,7 @@
}
/* Action 0x0C */
-static void GRFComment(byte *buf, int len)
+static void GRFComment(byte *buf, size_t len)
{
/* <0C> [<ignored...>]
*
@@ -4175,13 +4175,13 @@
if (len == 1) return;
- int text_len = len - 1;
+ size_t text_len = len - 1;
const char *text = (const char*)(buf + 1);
grfmsg(2, "GRFComment: %.*s", text_len, text);
}
/* Action 0x0D (GLS_SAFETYSCAN) */
-static void SafeParamSet(byte *buf, int len)
+static void SafeParamSet(byte *buf, size_t len)
{
if (!check_length(len, 5, "SafeParamSet")) return;
buf++;
@@ -4313,7 +4313,7 @@
/* Action 0x0D */
-static void ParamSet(byte *buf, int len)
+static void ParamSet(byte *buf, size_t len)
{
/* <0D> <target> <operation> <source1> <source2> [<data>]
*
@@ -4604,7 +4604,7 @@
}
/* Action 0x0E (GLS_SAFETYSCAN) */
-static void SafeGRFInhibit(byte *buf, int len)
+static void SafeGRFInhibit(byte *buf, size_t len)
{
/* <0E> <num> <grfids...>
*
@@ -4632,7 +4632,7 @@
}
/* Action 0x0E */
-static void GRFInhibit(byte *buf, int len)
+static void GRFInhibit(byte *buf, size_t len)
{
/* <0E> <num> <grfids...>
*
@@ -4657,7 +4657,7 @@
}
/* Action 0x0F */
-static void FeatureTownName(byte *buf, int len)
+static void FeatureTownName(byte *buf, size_t len)
{
/* <0F> <id> <style-name> <num-parts> <parts>
*
@@ -4757,7 +4757,7 @@
}
/* Action 0x10 */
-static void DefineGotoLabel(byte *buf, int len)
+static void DefineGotoLabel(byte *buf, size_t len)
{
/* <10> <label> [<comment>]
*
@@ -4787,7 +4787,7 @@
}
/* Action 0x11 */
-static void GRFSound(byte *buf, int len)
+static void GRFSound(byte *buf, size_t len)
{
/* <11> <num>
*
@@ -4804,7 +4804,7 @@
}
/* Action 0x11 (SKIP) */
-static void SkipAct11(byte *buf, int len)
+static void SkipAct11(byte *buf, size_t len)
{
/* <11> <num>
*
@@ -4930,7 +4930,7 @@
}
/* Action 0x12 */
-static void LoadFontGlyph(byte *buf, int len)
+static void LoadFontGlyph(byte *buf, size_t len)
{
/* <12> <num_def> <font_size> <num_char> <base_char>
*
@@ -4962,7 +4962,7 @@
}
/* Action 0x12 (SKIP) */
-static void SkipAct12(byte *buf, int len)
+static void SkipAct12(byte *buf, size_t len)
{
/* <12> <num_def> <font_size> <num_char> <base_char>
*
@@ -4992,7 +4992,7 @@
}
/* Action 0x13 */
-static void TranslateGRFStrings(byte *buf, int len)
+static void TranslateGRFStrings(byte *buf, size_t len)
{
/* <13> <grfid> <num-ent> <offset> <text...>
*
@@ -5086,7 +5086,7 @@
/* Used during safety scan on unsafe actions */
-static void GRFUnsafe(byte *buf, int len)
+static void GRFUnsafe(byte *buf, size_t len)
{
SetBit(_cur_grfconfig->flags, GCF_UNSAFE);
--- a/src/newgrf_config.cpp Thu May 08 13:06:12 2008 +0000
+++ b/src/newgrf_config.cpp Thu May 08 13:20:54 2008 +0000
@@ -309,7 +309,7 @@
}
/* Scan a path for NewGRFs */
-static uint ScanPath(const char *path, int basepath_length)
+static uint ScanPath(const char *path, size_t basepath_length)
{
extern bool FiosIsValidFile(const char *path, const struct dirent *ent, struct stat *sb);
--- a/src/settings.cpp Thu May 08 13:06:12 2008 +0000
+++ b/src/settings.cpp Thu May 08 13:20:54 2008 +0000
@@ -78,18 +78,18 @@
typedef void SettingDescProcList(IniFile *ini, const char *grpname, char **list, uint len, SettingListCallbackProc proc);
static void pool_init(SettingsMemoryPool **pool);
-static void *pool_alloc(SettingsMemoryPool **pool, uint size);
-static void *pool_strdup(SettingsMemoryPool **pool, const char *mem, uint size);
+static void *pool_alloc(SettingsMemoryPool **pool, size_t size);
+static void *pool_strdup(SettingsMemoryPool **pool, const char *mem, size_t size);
static void pool_free(SettingsMemoryPool **pool);
static bool IsSignedVarMemType(VarType vt);
struct SettingsMemoryPool {
- uint pos, size;
+ size_t pos, size;
SettingsMemoryPool *next;
byte mem[1];
};
-static SettingsMemoryPool *pool_new(uint minsize)
+static SettingsMemoryPool *pool_new(size_t minsize)
{
SettingsMemoryPool *p;
if (minsize < 4096 - 12) minsize = 4096 - 12;
@@ -106,9 +106,9 @@
*pool = pool_new(0);
}
-static void *pool_alloc(SettingsMemoryPool **pool, uint size)
+static void *pool_alloc(SettingsMemoryPool **pool, size_t size)
{
- uint pos;
+ size_t pos;
SettingsMemoryPool *p = *pool;
size = Align(size, sizeof(void*));
@@ -129,7 +129,7 @@
return p->mem + pos;
}
-static void *pool_strdup(SettingsMemoryPool **pool, const char *mem, uint size)
+static void *pool_strdup(SettingsMemoryPool **pool, const char *mem, size_t size)
{
byte *p = (byte*)pool_alloc(pool, size + 1);
p[size] = 0;
@@ -186,7 +186,7 @@
}
/** allocate an ini group object */
-static IniGroup *ini_group_alloc(IniFile *ini, const char *grpt, int len)
+static IniGroup *ini_group_alloc(IniFile *ini, const char *grpt, size_t len)
{
IniGroup *grp = (IniGroup*)pool_alloc(&ini->pool, sizeof(IniGroup));
grp->ini = ini;
@@ -205,7 +205,7 @@
return grp;
}
-static IniItem *ini_item_alloc(IniGroup *group, const char *name, int len)
+static IniItem *ini_item_alloc(IniGroup *group, const char *name, size_t len)
{
IniItem *item = (IniItem*)pool_alloc(&group->ini->pool, sizeof(IniItem));
item->name = (char*)pool_strdup(&group->ini->pool, name, len);
@@ -324,7 +324,7 @@
}
/** lookup a group or make a new one */
-static IniGroup *ini_getgroup(IniFile *ini, const char *name, int len)
+static IniGroup *ini_getgroup(IniFile *ini, const char *name, size_t len)
{
IniGroup *group;
@@ -345,7 +345,7 @@
static IniItem *ini_getitem(IniGroup *group, const char *name, bool create)
{
IniItem *item;
- uint len = strlen(name);
+ size_t len = strlen(name);
for (item = group->item; item; item = item->next)
if (strcmp(item->name, name) == 0) return item;
@@ -404,7 +404,7 @@
* @param one the current value of the setting for which a value needs found
* @param onelen force calculation of the *one parameter
* @return the integer index of the full-list, or -1 if not found */
-static int lookup_oneofmany(const char *many, const char *one, int onelen)
+static int lookup_oneofmany(const char *many, const char *one, size_t onelen)
{
const char *s;
int idx;
--- a/src/spritecache.cpp Thu May 08 13:06:12 2008 +0000
+++ b/src/spritecache.cpp Thu May 08 13:20:54 2008 +0000
@@ -61,7 +61,7 @@
struct MemBlock {
- uint32 size;
+ size_t size;
byte data[VARARRAY_SIZE];
};
--- a/src/string_func.h Thu May 08 13:06:12 2008 +0000
+++ b/src/string_func.h Thu May 08 13:20:54 2008 +0000
@@ -45,10 +45,10 @@
/** Get the length of a string, within a limited buffer */
-static inline int ttd_strnlen(const char *str, int maxlen)
+static inline size_t ttd_strnlen(const char *str, size_t maxlen)
{
const char *t;
- for (t = str; *t != '\0' && t - str < maxlen; t++) {}
+ for (t = str; *t != '\0' && (size_t)(t - str) < maxlen; t++) {}
return t - str;
}
--- a/src/vehicle.cpp Thu May 08 13:06:12 2008 +0000
+++ b/src/vehicle.cpp Thu May 08 13:20:54 2008 +0000
@@ -2691,7 +2691,7 @@
{
int length = CountVehiclesInChain(v);
- uint cargo_packages_count = 1;
+ size_t cargo_packages_count = 1;
for (const Vehicle *v_count = v; v_count != NULL; v_count=v_count->Next()) {
/* Now we count how many cargo packets we need to store.
* We started with an offset by one because we also need an end of array marker. */
--- a/src/win32.cpp Thu May 08 13:06:12 2008 +0000
+++ b/src/win32.cpp Thu May 08 13:20:54 2008 +0000
@@ -206,16 +206,15 @@
HMODULE modules[100];
DWORD needed;
BOOL res;
- int count, i;
HANDLE proc = OpenProcess(PROCESS_ALL_ACCESS, FALSE, GetCurrentProcessId());
if (proc != NULL) {
res = EnumProcessModules(proc, modules, sizeof(modules), &needed);
CloseHandle(proc);
if (res) {
- count = min(needed / sizeof(HMODULE), lengthof(modules));
+ size_t count = min(needed / sizeof(HMODULE), lengthof(modules));
- for (i = 0; i != count; i++) output = PrintModuleInfo(output, modules[i]);
+ for (size_t i = 0; i != count; i++) output = PrintModuleInfo(output, modules[i]);
return output;
}
}