Hallo Uwe: > thanks a lot for your patches and issue reports (and patience)!
Thank you for checking the changes in! How come PulseView ends up including Python headers through sigrokdecode.h? I saw this in libsigrokdecode : #include <Python.h> /* First, so we avoid a _POSIX_C_SOURCE warning. */ This "must be included first" rule propagates to sigrokdecode.h and then to anything that includes it, so I find it rather annoying. Imagine if every include file wanted to the be first one. I guess this is a problem in Python, the symbols they need to define should actually be defined at makefile level, but that's something we cannot fix in this project. However, I was hoping that, if the GUI is just using the Python decoders, they would be encapsulated in libsigrokdecode so that the GUI itself wouldn't need to know anything about Python. We may even want to write decoders in another language in the future. By the way, I suspect sigrok-cli is broken, I'm getting this error message: CC sigrok-cli.o sigrok-cli.c: In function ‘load_input_file_format’: sigrok-cli.c:1192:3: error: too few arguments to function ‘in->format->init’ Cheers, rdiez ------------------------------------------------------------------------------ Everyone hates slow websites. So do we. Make your web apps faster with AppDynamics Download AppDynamics Lite for free today: http://p.sf.net/sfu/appdyn_d2d_feb _______________________________________________ sigrok-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/sigrok-devel
