search

cloudwu / bgfxidl

An IDL for bgfx
28 stars 5 forks source link

IDL docs. #19

Closed bkaradzic closed 5 years ago

bkaradzic commented 5 years ago

There is issue with class codegen. It looks like:

    /// Set scissor for draw primitive.

    /// 

    /// @remark

    /// To scissor for all primitives in view see `bgfx::setViewScissor`.

    /// 

    ///

    /// @param[in] _x Position x from the left corner of the window.

    /// @param[in] _y Position y from the top corner of the window.

    /// @param[in] _width Width of view scissor region.

    /// @param[in] _height Height of view scissor region.

    ///

    /// @returns Scissor cache index.

    ///

    /// @attention C99 equivalent is `bgfx_encoder_set_scissor`.

    ///

    uint16_t setScissor(

          uint16_t _x

        , uint16_t _y

        , uint16_t _width

        , uint16_t _height

        );
cloudwu commented 5 years ago

It's correct in lua 5.3.5 .

It's a similar string.gmatch issue . I change a different pattern to avoid it. https://github.com/cloudwu/bgfxidl/commit/8017fb1e1a9d4af3364e8a0d775171239626bc72

bkaradzic commented 5 years ago

I was thinking of just upgrading Lua in GENie, but I think there was some issue with newer Lua before. Just can't remember it, and don't want to break GENie by update...

cloudwu commented 5 years ago

There was a discussion about this issue in lua mailling list http://lua-users.org/lists/lua-l/2013-04/msg00812.html