Author: esr
Date: Wed Apr 18 13:28:40 2007
New Revision: 16896
URL: http://svn.gna.org/viewcvs/wesnoth?rev=16896&view=rev
Log:
Merge back in most of mog's commit, Mordante thinks he can fix it.
Modified:
trunk/src/builder.cpp
trunk/src/terrain_translation.cpp
trunk/src/terrain_translation.hpp
Modified: trunk/src/builder.cpp
URL:
http://svn.gna.org/viewcvs/wesnoth/trunk/src/builder.cpp?rev=16896&r1=16895&r2=16896&view=diff
==============================================================================
--- trunk/src/builder.cpp (original)
+++ trunk/src/builder.cpp Wed Apr 18 13:28:40 2007
@@ -545,7 +545,7 @@
void terrain_builder::add_constraints(terrain_builder::constraint_set
&constraints, const gamemap::location& loc, const config& cfg, const config&
global_images)
{
- add_constraints(constraints, loc, t_translation::t_match(cfg["type"]),
global_images);
+ add_constraints(constraints, loc, t_translation::t_match(cfg["type"],
t_translation::WILDCARD), global_images);
terrain_constraint& constraint = constraints[loc];
@@ -1009,7 +1009,7 @@
if(cons != rule->second.constraints.end()) {
adjacent_types[i] =
cons->second.terrain_types_match.terrain;
} else {
- adjacent_types[i] =
t_translation::read_list("", -1, t_translation::T_FORMAT_STRING);
+ adjacent_types[i] =
t_translation::read_list("", -1, t_translation::T_FORMAT_STRING,
t_translation::WILDCARD);
}
}
Modified: trunk/src/terrain_translation.cpp
URL:
http://svn.gna.org/viewcvs/wesnoth/trunk/src/terrain_translation.cpp?rev=16896&r1=16895&r2=16896&view=diff
==============================================================================
--- trunk/src/terrain_translation.cpp (original)
+++ trunk/src/terrain_translation.cpp Wed Apr 18 13:28:40 2007
@@ -151,23 +151,23 @@
/***************************************************************************************/
-const t_letter VOID_TERRAIN = string_to_number_("_s");
-const t_letter FOGGED = string_to_number_("_f");
-
-const t_letter HUMAN_CASTLE = string_to_number_("Ch");
-const t_letter HUMAN_KEEP = string_to_number_("Kh");
-const t_letter SHALLOW_WATER = string_to_number_("Ww");
-const t_letter DEEP_WATER = string_to_number_("Wo");
-const t_letter GRASS_LAND = string_to_number_("Gg");
-const t_letter FOREST = string_to_number_("Ff");
-const t_letter MOUNTAIN = string_to_number_("Mm");
-const t_letter HILL = string_to_number_("Hh");
-
-const t_letter CAVE_WALL = string_to_number_("Xu");
-const t_letter CAVE = string_to_number_("Uu");
-const t_letter UNDERGROUND_VILLAGE = string_to_number_("Vu");
-const t_letter DWARVEN_CASTLE = string_to_number_("Cud");
-const t_letter DWARVEN_KEEP = string_to_number_("Kud");
+const t_letter VOID_TERRAIN = string_to_number_("_s", NO_LAYER);
+const t_letter FOGGED = string_to_number_("_f", NO_LAYER);
+
+const t_letter HUMAN_CASTLE = string_to_number_("Ch", NO_LAYER);
+const t_letter HUMAN_KEEP = string_to_number_("Kh", NO_LAYER);
+const t_letter SHALLOW_WATER = string_to_number_("Ww", NO_LAYER);
+const t_letter DEEP_WATER = string_to_number_("Wo", NO_LAYER);
+const t_letter GRASS_LAND = string_to_number_("Gg", NO_LAYER);
+const t_letter FOREST = string_to_number_("Ff", NO_LAYER);
+const t_letter MOUNTAIN = string_to_number_("Mm", NO_LAYER);
+const t_letter HILL = string_to_number_("Hh", NO_LAYER);
+
+const t_letter CAVE_WALL = string_to_number_("Xu", NO_LAYER);
+const t_letter CAVE = string_to_number_("Uu", NO_LAYER);
+const t_letter UNDERGROUND_VILLAGE = string_to_number_("Vu", NO_LAYER);
+const t_letter DWARVEN_CASTLE = string_to_number_("Cud", NO_LAYER);
+const t_letter DWARVEN_KEEP = string_to_number_("Kud", NO_LAYER);
const t_letter PLUS = string_to_number_("+");
const t_letter MINUS = string_to_number_("-");
@@ -177,7 +177,11 @@
/***************************************************************************************/
t_letter::t_letter(const std::string& b) :
- base(string_to_layer_(b)), overlay(0xFFFFFFFF)
+ base(string_to_layer_(b)), overlay(NO_LAYER)
+{}
+
+t_letter::t_letter(const std::string& b, const t_layer o) :
+ base(string_to_layer_(b)), overlay(o)
{}
t_letter::t_letter(const std::string& b, const std::string& o) :
@@ -188,8 +192,8 @@
has_wildcard(false), is_empty(true)
{}
-t_match::t_match(const std::string& str):
- terrain(t_translation::read_list(str, -1,
t_translation::T_FORMAT_STRING))
+t_match::t_match(const std::string& str, const t_layer filler):
+ terrain(t_translation::read_list(str, -1,
t_translation::T_FORMAT_STRING, filler))
{
mask.resize(terrain.size());
masked_terrain.resize(terrain.size());
@@ -216,12 +220,12 @@
}
}
-t_letter read_letter(const std::string& str, const int t_format)
+t_letter read_letter(const std::string& str, const int t_format, const t_layer
filler)
{
#ifdef TERRAIN_TRANSLATION_COMPATIBLE
if(t_format == T_FORMAT_STRING ||
(t_format == T_FORMAT_AUTO && map_format_ == 2)) {
- return string_to_number_(str);
+ return string_to_number_(str, filler);
} else if(t_format == T_FORMAT_LETTER ||
(t_format == T_FORMAT_AUTO && map_format_ == 1)) {
@@ -231,7 +235,7 @@
throw error("Invalid case in read_letter");
}
#else
- return string_to_number_(str);
+ return string_to_number_(str, filler);
#endif
}
@@ -240,12 +244,12 @@
return number_to_string_(letter);
}
-t_list read_list(const std::string& str, const int separated, const int
t_format)
+t_list read_list(const std::string& str, const int separated, const int
t_format, const t_layer filler)
{
#ifdef TERRAIN_TRANSLATION_COMPATIBLE
if(t_format == T_FORMAT_STRING ||
(t_format == T_FORMAT_AUTO && map_format_ == 2)) {
- return string_to_vector_(str);
+ return string_to_vector_(str, filler);
} else if(t_format == T_FORMAT_LETTER ||
(t_format == T_FORMAT_AUTO && map_format_ == 1)) {
@@ -255,7 +259,7 @@
throw error("Invalid case in read_list");
}
#else
- return string_to_vector_(str);
+ return string_to_vector_(str, filler);
#endif
}
@@ -321,8 +325,9 @@
const std::string terrain = str.substr(offset, pos_separator -
offset);
// process the chunk
- int starting_position = -1;
- const t_letter tile = string_to_number_(terrain,
starting_position);
+ int starting_position = -1;
+ // the gamemap never has a wildcard
+ const t_letter tile = string_to_number_(terrain,
starting_position, NO_LAYER);
// add to the resulting starting position
if(starting_position != -1) {
@@ -502,12 +507,12 @@
return result;
}
- if(src_has_wildcard && src.overlay == 0 && itor->overlay ==
0xFFFFFFFF &&
+ if(src_has_wildcard && src.overlay == 0 && itor->overlay ==
NO_LAYER &&
((itor->base & src_mask.base) ==
masked_src.base )) {
return result;
}
- if(dest_has_wildcard && itor->overlay == 0 && src.overlay ==
0xFFFFFFFF &&
+ if(dest_has_wildcard && itor->overlay == 0 && src.overlay ==
NO_LAYER &&
((src.base & dest_mask.base) ==
masked_dest.base)) {
return result;
}
@@ -575,7 +580,7 @@
// does the source have a wildcard and an empty overlay and the
destination
// no overlay, we need to check the part base for a match
- if(src_has_wildcard && src.overlay == 0 &&
terrain_itor->overlay == 0xFFFFFFFF &&
+ if(src_has_wildcard && src.overlay == 0 &&
terrain_itor->overlay == NO_LAYER &&
((terrain_itor->base & src_mask.base) ==
masked_src.base )) {
return result;
}
@@ -584,7 +589,7 @@
// no overlay, we need to check the part base for a match
// NOTE the has_wildcard(*terrain_itor) is expensive so move
the test to
// later in the line
- if(terrain_itor->overlay == 0 && src.overlay == 0xFFFFFFFF &&
has_wildcard(*terrain_itor) &&
+ if(terrain_itor->overlay == 0 && src.overlay == NO_LAYER &&
has_wildcard(*terrain_itor) &&
((src.base & dest.mask[i].base) ==
dest.masked_terrain[i].base)) {
return result;
}
@@ -597,10 +602,10 @@
bool has_wildcard(const t_letter& letter)
{
- if(letter.overlay == 0xFFFFFFFF) {
- return get_layer_mask_(letter.base) != 0xFFFFFFFF;
+ if(letter.overlay == NO_LAYER) {
+ return get_layer_mask_(letter.base) != NO_LAYER;
} else {
- return get_layer_mask_(letter.base) != 0xFFFFFFFF ||
get_layer_mask_(letter.overlay) != 0xFFFFFFFF;
+ return get_layer_mask_(letter.base) != NO_LAYER ||
get_layer_mask_(letter.overlay) != NO_LAYER;
}
}
@@ -721,7 +726,7 @@
static t_list string_to_vector_(const std::string& str, const bool
convert_eol, const int separated)
{
// only used here so define here
- const t_letter EOL(7, 0xFFFFFFFF);
+ const t_letter EOL(7, NO_LAYER);
bool last_eol = false;
t_list result;
@@ -878,7 +883,7 @@
const std::string terrain = str.substr(offset, pos_separator -
offset);
// process the chunk
- const t_letter tile = string_to_number_(terrain);
+ const t_letter tile = string_to_number_(terrain, filler);
// add the resulting terrain number
result.push_back(tile);
@@ -894,7 +899,7 @@
return result;
}
-inline Uint32 get_layer_mask_(Uint32 terrain)
+inline t_layer get_layer_mask_(t_layer terrain)
{
// test for the star 0x2A in every postion and return the
// appropriate mask
@@ -983,7 +988,7 @@
const std::string overlay_str(str, offset + 1, str.size());
result = t_letter(base_str, overlay_str);
} else {
- result = t_letter(str);
+ result = t_letter(str, filler);
}
#ifndef TERRAIN_TRANSLATION_COMPATIBLE
@@ -1012,7 +1017,7 @@
letter[2] = ((terrain.base & 0x0000FF00) >> 8);
letter[3] = (terrain.base & 0x000000FF);
- if(terrain.overlay != 0xFFFFFFFF) {
+ if(terrain.overlay != NO_LAYER) {
letter[4] = '^'; //the layer separator
letter[5] = ((terrain.overlay & 0xFF000000) >> 24);
letter[6] = ((terrain.overlay & 0x00FF0000) >> 16);
Modified: trunk/src/terrain_translation.hpp
URL:
http://svn.gna.org/viewcvs/wesnoth/trunk/src/terrain_translation.hpp?rev=16896&r1=16895&r2=16896&view=diff
==============================================================================
--- trunk/src/terrain_translation.hpp (original)
+++ trunk/src/terrain_translation.hpp Wed Apr 18 13:28:40 2007
@@ -35,6 +35,10 @@
namespace t_translation {
+ typedef Uint32 t_layer;
+ const t_layer WILDCARD = 0x2A000000;
+ const t_layer NO_LAYER = 0xFFFFFFFF;
+
// The definitions for a terrain
/**
* A terrain string which is converted to a terrain is a string with 1
or 2 layers
@@ -45,11 +49,12 @@
struct t_letter {
t_letter(const std::string& b);
t_letter(const std::string& b, const std::string& o);
- t_letter(const Uint32& b, const Uint32& o) : base(b),
overlay(o) {};
- t_letter() : base(0), overlay(0xFFFFFFFF) {}
-
- Uint32 base;
- Uint32 overlay;
+ t_letter(const std::string& b, const t_layer o);
+ t_letter(const t_layer& b, const t_layer& o) : base(b),
overlay(o) {};
+ t_letter() : base(0), overlay(NO_LAYER) {}
+
+ t_layer base;
+ t_layer overlay;
};
const t_letter NONE_TERRAIN = t_letter();
@@ -80,7 +85,7 @@
*/
struct t_match{
t_match();
- t_match(const std::string& str);
+ t_match(const std::string& str, const t_layer filler =
NO_LAYER);
t_match(const t_letter& letter);
t_list terrain;
@@ -159,10 +164,11 @@
* is the base terrain, the second
the overlay terrain.
*
* @param t_format The format to read
+ * @param filler if there's no layer this value will be used as
the second layer
*
* @return A single terrain letter
*/
- t_letter read_letter(const std::string& str, const int t_format);
+ t_letter read_letter(const std::string& str, const int t_format, const
t_layer filler = NO_LAYER);
/**
* Writes a single letter to a string.
@@ -189,10 +195,11 @@
* 0 = no
* 1 = yes
* @param t_format The format to read.
+ * @param filler if there's no layer this value will be used as
the second layer
*
* @returns A vector which contains the letters
found in the string
*/
- t_list read_list(const std::string& str, const int separated, const
int t_format);
+ t_list read_list(const std::string& str, const int separated, const
int t_format, const t_layer filler = NO_LAYER);
/**
* Writes a list of terrains to a string, only writes the new format.
@@ -324,8 +331,8 @@
// these terrain letters are in the builder format, and
// not usable in other parts of the engine
- const Uint32 TB_STAR = '*' << 24; //it can be assumed this is the
equivalent of STAR
- const Uint32 TB_DOT = '.' << 24;
+ const t_layer TB_STAR = '*' << 24; //it can be assumed this is the
equivalent of STAR
+ const t_layer TB_DOT = '.' << 24;
/**
* Reads a builder map, a builder map differs much from a normal map
hence
_______________________________________________
Wesnoth-commits mailing list
[email protected]
https://mail.gna.org/listinfo/wesnoth-commits
