On 07/07/2015 at 09:55, xxxxxxxx wrote:
Hey,
The crash is exactly in the line String name = data.GetString()
I have other code lines in there, the posted code was a simplification. I actually read other stuff from the same tag, before the crash.
Interestingly enough, if I have a breakpoint in the next line (after the GetString()) it always crashes. If I do not have any breakpoint near this code, it's very hard to get a crash here (but it happens sometimes).
On 07/07/2015 at 10:52, xxxxxxxx wrote:
Hi,
About your question of Int32, I'd recommend reading this:
https://en.wikipedia.org/wiki/Integer_(computer_science)
On 07/07/2015 at 10:59, xxxxxxxx wrote:
Originally posted by xxxxxxxx
Hi,
About your question of Int32, I'd recommend reading this:
https://en.wikipedia.org/wiki/Integer_(computer_science)
Ok, I kind of deserve that for not being clear enough!
Well, What I meant was, "what is the meaning of and Int32 value as a result of the Get TEXTURETAG_RESTRICTION".
I know what an Integer is, what I don't know is what its value represents here.
On 07/07/2015 at 11:40, xxxxxxxx wrote:
Hi,
Okay, now we know that 
I wasn't able to find anything about that on the docs.
Could it be an enum value? Just my thoughts here.
On 07/07/2015 at 13:43, xxxxxxxx wrote:
Here are my thoughts on this:
1. Do not assume that the first tag is a Texture tag. And you might be like well it is the first and maybe only tag on the object. And you would be incorrect. Polygon objects also have two hidden tags: a Point tag and a Polygon tag. Verify that you are getting the proper tag type. Would be better to iterate through the object's tags until you find a texture tag. There may be more than one or none.
if (tag && tag->IsInstanceOf(Ttexture) && tag->GetParameter(TEXTURETAG_RESTRICTION, data))
2. TEXTURETAG_RESTRICTION is defined as a String description in the Ttexture.res file. Therefore, it should *always* return a String. That is why I mention point 1. Could be that you are requesting a parameter for a tag that isn't a Texture tag and thus the issues.
On 07/07/2015 at 13:52, xxxxxxxx wrote:
Hi,
So I thought that simplifying the code was the best approach to make myself clear, but I was wrong. So here is the full code until the problematic line:
BaseObject *searchOp = (PolygonObject* )op;
while (!found && searchOp != nullptr)
{
BaseTag* btag = searchOp->GetFirstTag();
for (; btag; btag = (BaseTag* )btag->GetNext())
{
if (btag->GetType() == Ttexture)
{
if (btag->GetParameter(TEXTURETAG_MATERIAL, data))
{
MaterialSelection m;
m.mat = (AlienMaterial* )data.GetLink();
found = true;
if (m.mat != nullptr && std::find(std::begin(materials), std::end(materials), m) == end(materials))
{
if (btag->GetParameter(TEXTURETAG_RESTRICTION, data))
{
String name = data.GetString();
I think I have all the needed checks, however, if I check the "data" type (data.GetType()) it returns Int32, and the the data.GetString() crashes.
On 07/07/2015 at 15:04, xxxxxxxx wrote:
data.GetType() does indeed return Int32. It is a value representing the 'type' of data for the description. It should be DA_STRING.
if (data.GetType() != DA_STRING) //then something is wrong
Otherwise, I don't see anything wrong with the code (except the possible issues using the STL).
On 07/07/2015 at 15:57, xxxxxxxx wrote:
That is very odd. Possible memory corruption? Is the restriction empty when it crashes or it doesn't matter either way? There has to be some outside factor for the data type to return DA_LONG when it has to be DA_STRING.
On 07/07/2015 at 16:30, xxxxxxxx wrote:
It could be empty yes, because all variables are null, and the type could just be the initial value, or even memory trash.
Either way, does it make sense to crash in situations like this? Also, why the GetParameter returns true, if the data is empty?
I check for type now, to make sure it does not read invalid stuff, but it was not needed before.