dariocc / fenixlib Goto Github PK

Current constructors:
public GlyphInfo ( int width, int height, int yOffset, int fileOffset )

public GlyphInfo ( int width, int height, int xOffset,
    int yOffset, int xAdvance, int yAdvance, int fileOffset )

take many arguments of ints, which makes the client code brittle and difficult to follow (it already happened that there order of the arguments was wrong).

A builder object + fluent API will make it more difficult to break.

It seems that the versions of MonoDevelop / Xamarin that I have tested do not have NUnit 3 support. It is however possible to run the test by using NUnit's CLI.

After #16 fix.

Adding extension methods to perform the mentioned conversions will make it nicer to interact between System.Drawing and FenixLib:

For example:

bitmap.ToGraphic ();
graphic.ToBitmap ();

Stream.Position cannot be used with Gz compressed streams, therefore the program fails.
Tested only in Mono.

Replace the Width * height * Depth / 8 calculation for all encoders / decoders with the NativeFormat.CalculatePixelBufferBytes.

oeste.zip

Add verification for expectation that method / properties are effectively returning the same reference / value than sprite pased to the constructor.

The reason is the gamma section after the palette, which is currently filled with 0.