Fixed and committed to the SVN. Thanks for the report.

Best,
Scuri


Em qui, 6 de dez de 2018 às 15:48, Matthew Kennedy <burnsid...@gmail.com>
escreveu:

> Hi I've attached the complete classname/cbname/cbspec as CSV. In
> summary, it looks like only DRAGDATA_CB and DROPDATA_CB are using "C"
> when they should using "V".
>
> The report was generated on Linux, from IUP:
>
> "3.25"
> "2018/05/28"
> 325000 (19 bits, #x4F588)
>
> 92 matches for "\(DRAG\|DROP\)DATA_CB" in buffer: report.txt
>       4:animatedlabel,DRAGDATA_CB,sCi
>       6:animatedlabel,DROPDATA_CB,sCiii
>      19:backgroundbox,DRAGDATA_CB,sCi
>      21:backgroundbox,DROPDATA_CB,sCiii
>      68:canvas,DRAGDATA_CB,sCi
>      70:canvas,DROPDATA_CB,sCiii
>      95:colorbar,DRAGDATA_CB,sCi
>      97:colorbar,DROPDATA_CB,sCiii
>     125:colorbrowser,DRAGDATA_CB,sCi
>     128:colorbrowser,DROPDATA_CB,sCiii
>     153:colordlg,DRAGDATA_CB,sCi
>     155:colordlg,DROPDATA_CB,sCiii
>     184:dial,DRAGDATA_CB,sCi
>     186:dial,DROPDATA_CB,sCiii
>     211:dialog,DRAGDATA_CB,sCi
>     213:dialog,DROPDATA_CB,sCiii
>     234:dropbutton,DRAGDATA_CB,sCi
>     236:dropbutton,DROPDATA_CB,sCiii
>     270:filedlg,DRAGDATA_CB,sCi
>     272:filedlg,DROPDATA_CB,sCiii
>     294:flatbutton,DRAGDATA_CB,sCi
>     296:flatbutton,DROPDATA_CB,sCiii
>     326:flatframe,DRAGDATA_CB,sCi
>     328:flatframe,DROPDATA_CB,sCiii
>     352:flatlabel,DRAGDATA_CB,sCi
>     354:flatlabel,DROPDATA_CB,sCiii
>     378:flatscrollbox,DRAGDATA_CB,sCi
>     380:flatscrollbox,DROPDATA_CB,sCiii
>     406:flatseparator,DRAGDATA_CB,sCi
>     408:flatseparator,DROPDATA_CB,sCiii
>     432:flattabs,DRAGDATA_CB,sCi
>     434:flattabs,DROPDATA_CB,sCiii
>     468:flattoggle,DRAGDATA_CB,sCi
>     470:flattoggle,DROPDATA_CB,sCiii
>     499:fontdlg,DRAGDATA_CB,sCi
>     501:fontdlg,DROPDATA_CB,sCiii
>     525:gauge,DRAGDATA_CB,sCi
>     527:gauge,DROPDATA_CB,sCiii
>     556:label,DRAGDATA_CB,sCi
>     558:label,DROPDATA_CB,sCiii
>     570:link,DRAGDATA_CB,sCi
>     572:link,DROPDATA_CB,sCiii
>     587:list,DRAGDATA_CB,sCi
>     590:list,DROPDATA_CB,sCiii
>     615:messagedlg,DRAGDATA_CB,sCi
>     617:messagedlg,DROPDATA_CB,sCiii
>     639:multiline,DRAGDATA_CB,sCi
>     641:multiline,DROPDATA_CB,sCiii
>     666:progressdlg,DRAGDATA_CB,sCi
>     668:progressdlg,DROPDATA_CB,sCiii
>     689:scrollbox,DRAGDATA_CB,sCi
>     691:scrollbox,DROPDATA_CB,sCiii
>     739:text,DRAGDATA_CB,sCi
>     741:text,DROPDATA_CB,sCiii
>     775:tree,DRAGDATA_CB,sCi
>     778:tree,DROPDATA_CB,sCiii
>     820:matrixlist,DRAGDATA_CB,sCi
>     824:matrixlist,DROPDATA_CB,sCiii
>     888:matrixex,DRAGDATA_CB,sCi
>     892:matrixex,DROPDATA_CB,sCiii
>     950:matrix,DRAGDATA_CB,sCi
>     954:matrix,DROPDATA_CB,sCiii
>    1005:cells,DRAGDATA_CB,sCi
>    1008:cells,DROPDATA_CB,sCiii
>    1041:glcanvas,DRAGDATA_CB,sCi
>    1043:glcanvas,DROPDATA_CB,sCiii
>    1068:glbackgroundbox,DRAGDATA_CB,sCi
>    1070:glbackgroundbox,DROPDATA_CB,sCiii
>    1226:glcanvasbox,DRAGDATA_CB,sCi
>    1228:glcanvasbox,DROPDATA_CB,sCiii
>    1253:glcanvas,DRAGDATA_CB,sCi
>    1255:glcanvas,DROPDATA_CB,sCiii
>    1291:glbackgroundbox,DRAGDATA_CB,sCi
>    1293:glbackgroundbox,DROPDATA_CB,sCiii
>    1325:plot,DRAGDATA_CB,sCi
>    1328:plot,DROPDATA_CB,sCiii
>    1365:glcanvas,DRAGDATA_CB,sCi
>    1367:glcanvas,DROPDATA_CB,sCiii
>    1392:glbackgroundbox,DRAGDATA_CB,sCi
>    1394:glbackgroundbox,DROPDATA_CB,sCiii
>    1419:mglplot,DRAGDATA_CB,sCi
>    1421:mglplot,DROPDATA_CB,sCiii
>    1448:mgllabel,DRAGDATA_CB,sCi
>    1450:mgllabel,DROPDATA_CB,sCiii
>    1477:glcanvas,DRAGDATA_CB,sCi
>    1479:glcanvas,DROPDATA_CB,sCiii
>    1504:glbackgroundbox,DRAGDATA_CB,sCi
>    1506:glbackgroundbox,DROPDATA_CB,sCiii
>    1533:scintilladlg,DRAGDATA_CB,sCi
>    1535:scintilladlg,DROPDATA_CB,sCiii
>    1566:scintilla,DRAGDATA_CB,sCi
>    1568:scintilla,DROPDATA_CB,sCiii
> On Thu, Dec 6, 2018 at 10:29 AM Antonio Scuri <antonio.sc...@gmail.com>
> wrote:
> >
> >   That's probably a bug too. I'll wait for your report then.
> >
> > Thanks,
> > Scuri
> >
> >
> > Em qui, 6 de dez de 2018 às 14:13, Matthew Kennedy <burnsid...@gmail.com>
> escreveu:
> >>
> >> Thanks for the fast reply! BTW, I did not see any other instances of
> >> lower-case v when building the API. However, I think there are
> >> instances of IUP using "C" when it should use "V" though. e.g. In
> >> animatedlabel, DROPDATA_CB is defined as "sCiii", but the "C" is
> >> really a void *, so probably should be "sViii".
> >>
> >> int function(Ihandle* ih, char* type, void* data, int size, int x, int
> y) [in C]
> >>
> >> I'll report back on that later today/tomorrow though. I can produce a
> >> list of class name, callback name, callback format/spec triples pretty
> >> easily now.
> >>
> >> Matt
> >> On Thu, Dec 6, 2018 at 6:10 AM Antonio Scuri <antonio.sc...@gmail.com>
> wrote:
> >> >
> >> >   Fixed and committed to the SVN. Thanks for reporting.
> >> >
> >> > Best,
> >> > Scuri
> >> >
> >> >
> >> > Em qui, 6 de dez de 2018 às 02:26, Matthew Kennedy <
> burnsid...@gmail.com> escreveu:
> >> >>
> >> >> Hi, Was working on some automated language bindings for IUP and ran
> >> >> into a problem with matrixlist and matrix.
> >> >>
> >> >> In iup_matrixlist.c:
> >> >>
> >> >>   iupClassRegisterCallback(ic, "LISTDRAW_CB", "iiiiiiv");
> >> >>
> >> >> There is a (lower-case) "v", which is a character not mentioned in
> the
> >> >> list defined in iup_class.h (c, i, I, f, d, s, V and C). I think the
> >> >> callback format above should have been "iiiiiiC", since the callback
> >> >> described in the docs is:
> >> >>
> >> >>   int function (Ihandle *ih, int lin, int col, int x1, int x2, int
> y1,
> >> >> int y2, cdCanvas* cnv); [in C]
> >> >>
> >> >> I think similarly, in iup_matrix.c:
> >> >>
> >> >>   iupClassRegisterCallback(ic, "DRAW_CB", "iiiiiiv");
> >> >>
> >> >> int function(Ihandle *ih, int lin, int col, int x1, int x2, int y1,
> >> >> int y2, cdCanvas* cnv); [in C]
> >> >>
> >> >> should be "iiiiiiC" (also the iupcbs.h describes it as IFniiiiiiC).
> >> >>
> >> >> Matt
> >> >>
> >> >>
> >> >> _______________________________________________
> >> >> Iup-users mailing list
> >> >> Iup-users@lists.sourceforge.net
> >> >> https://lists.sourceforge.net/lists/listinfo/iup-users
> >> >
> >> > _______________________________________________
> >> > Iup-users mailing list
> >> > Iup-users@lists.sourceforge.net
> >> > https://lists.sourceforge.net/lists/listinfo/iup-users
> >>
> >>
> >> _______________________________________________
> >> Iup-users mailing list
> >> Iup-users@lists.sourceforge.net
> >> https://lists.sourceforge.net/lists/listinfo/iup-users
> >
> > _______________________________________________
> > Iup-users mailing list
> > Iup-users@lists.sourceforge.net
> > https://lists.sourceforge.net/lists/listinfo/iup-users
> _______________________________________________
> Iup-users mailing list
> Iup-users@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/iup-users
>
_______________________________________________
Iup-users mailing list
Iup-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/iup-users

Reply via email to