First version released

[Laurent]

Hi

I've finished a first version of the .Net binding, available on SVN.

There's one annoying bug : it crashes randomly on internal objects destructions. I suspect the code is not completely GC-safe, but this is also the first time I write C# code (and .Net in general).

So, if someone can point out the mistake (I think it's located in the ObjectBase class), let me know

Any other bug report or suggestion is also welcome, as I'm not an expert in C#. Especially regaring C# standards / idioms / philosophy.

One last thing : if someone could test it under Mono, just to make sure everything's ok (AFAIK Mono doesn't fully implements .Net 2.0).

[Ed Ropple]

There's one annoying bug : it crashes randomly on internal objects destructions. I suspect the code is not completely GC-safe, but this is also the first time I write C# code (and .Net in general).

So, if someone can point out the mistake (I think it's located in the ObjectBase class), let me know Smile

I haven't looked at the code yet, but if the bug occurs, you should be getting a stack trace and exception. Have a paste?

Any other bug report or suggestion is also welcome, as I'm not an expert in C#. Especially regaring C# standards / idioms / philosophy.

When I get some time I'll look, but you should probably keep in mind that it won't look all that much like the C++ version if we're trying to go with C# conventions. C# is a different (better, IMO--cleaner, more logical) beast.

One last thing : if someone could test it under Mono, just to make sure everything's ok (AFAIK Mono doesn't fully implements .Net 2.0).

They're in final release stages before Mono 2.0 comes out--in theory at least, they implement everything in the System namespace.

[Laurent]

I haven't looked at the code yet, but if the bug occurs, you should be getting a stack trace and exception. Have a paste?

It's ok, we found the bug on the french forum : it's caused by the GC collecting resource classes (like Image, Font, ...) in its own thread, causing any OpenGL call happening in Dispose() to fail.

When I get some time I'll look, but you should probably keep in mind that it won't look all that much like the C++ version if we're trying to go with C# conventions. C# is a different (better, IMO--cleaner, more logical) beast.

I know, and I've tried to stick as much as possible to the C# conventions, but as I'm not used to them I may have missed something

They're in final release stages before Mono 2.0 comes out--in theory at least, they implement everything in the System namespace.

So it should be ok, AFAIK the only thing I use from 2.0 is generic collections.

[vmilea]

Just noticed an X/Y mix-up in 'Vector2.cs':

operator-   : return new Vector2(v1.X - v2.X, v1.Y - v2.X);
operator+   : return new Vector2(v1.X + v2.X, v1.Y + v2.X);

[Laurent]

Thanks, it's now fixed.

Apart from that, did you test this binding ? Do you have any other good or bad feedback ?

[phenex]

I went through the French forum (I don't speak French), and I didn't find this bug mentioned. Hopefully this is news to you.

It seems that there is a problem with String2D.GetRect. I was able to get it to work (as far as I can tell) by changing GetRect to:

Code: [Select]

public FloatRect GetRect()
{
return (FloatRect)Marshal.PtrToStructure(sfString_GetRect(This), typeof(FloatRect));
}

I'll be fixing any bugs I can find as I go through it. Thanks for the wrapper.

[Laurent]

I can't see the difference between your code and mine (you're just using the other overload of the function, which does exactly the same thing). Can you explain what was wrong?

[phenex]

Was the previous version of GetRect working for you? It was not working for me. It would throw an ArgumentException inside GetRect, on the PtrToStructure call, indicating that Rect should not be a value class.

This version works for me. It creates the structure for you, so you don't need any code inside GetRect except that which I've already pasted.

[Laurent]

I still don't know why the first version didn't work, but I fixed it using your code.

Thanks

[Beliar]

RenderWindow.ConvertCoords(uint windowX, uint windowY) throws an NullReferenceException when no View has been set.

The easy way to fix it is to add

Code: [Select]


CurrentView = myDefaultView;

in the Initialize metehod.

Another (maybe better) way is to change RenderWindow.ConvertCoords(uint windowX, uint windowY) to this:

Code: [Select]


return ConvertCoords(windowX, windowY, CurrentView);


and the Getter of CurrentView:

Code: [Select]


get {
if(myCurrentView != null)
{
return myCurrentView;
}
else
{
return DefaultView;
}
}


Edit:
Oh and i tested it with Mono.
It compiles and runs without problems on windows.
It compiles and runs on Linux but i had problems which maybe are cause i used the VMWare image from the Mono website. Don't have any other Linux installed currently though.

[Laurent]

It's fixed, thanks.

I'm also glad to see that it works fine with Mono