C API Help

[blend2d]

C API Help

[kobalicek]

Thank you for your feedback. I think both issues are valid and I will fix them.

[speps]

A few things I've found while doing some bindings (still WIP):

• blImageCodecBuiltInCodecs: should either return const or actually do a weak assign onto an existing instance, in my binding that method looks like it'll have to be a special case when it really shouldn't be.
• blFontLoaderDataByFaceIndex: returns BLFontDataImpl*, seems like a big problem to me, leaking implementation details.

[kobalicek]

blFontLoaderDataByFaceIndex has been addressed - I think it makes more sense now.

[kobalicek]

blImageCodecBuiltInCodecs is gone as well. There are 2 new functions that can be used as a replacement:

• blImageCodecArrayInitBuiltInCodecs (initializer)
• blImageCodecArrayAssignBuiltInCodecs (use on existing valid instance)

Thank you again for the feedback, I hope these changes make sense and I would welcome more suggestions if you find anything!

[speps]

Awesome thanks a lot for this, great changes! I'll post my bindings somewhere once I'm done.

Side question: I'm currently using the blend2d-beta.zip link to download the source automatically when building my bindings, is it synced with the master branch?

[kobalicek]

The beta links are only snapshots that do not reflect the current git version. The idea is to make a new beta release when something changes and it's tested. For example today I will create a new snapshot as the API changed and I would like to propagate the change to users. I always bump the betaN number when releasing the new snapshot to make it more transparent when the snapshot is updated.

[martin31821]

@kobalicek Would you mind uploading your beta releases to github releases as well? This way users who are only watching releases will get notified whenever an API change was introduced.

[kobalicek]

Well I don't plan to add beta releases to github releases. I think it's too early to do this and I added beta source releases mostly for users that would like to evaluate Blend2D. I would recommend just using master branch for beta users that actually use the library.

[kobalicek]

I think it's plan to close this issue, open a new one for anything new.