Some suggestions on revision 2:
- my qwt include directory is /usr/include/qwt-qt4 (on Ubuntu 9.10
Karmic Koala) - one more reason to automatically look for libraries!
I also have to link with -lqwt-qt4
- in messagebufferinterface.h, I need to include <sys/time.h> for
'struct timeval', I don't know why you apparently don't.
- in C++, you declare structs without typedef, simply as:
1 | struct myStruct {
|
2 | // bla bla
|
3 | };
|
- mainwindow.h:49 -> 1. you don't usually declare variables in a header
file, because many source files may include them, and then all of them
have a separate variable, which then leads to a linker error.
2. why make a struct with a single entry?
- there are a huge lot of warnings, the compiler usually means them.
Good code doesn't produce warnings.
- write a README file which explains how to compile the code...
- SVN usually uses a directory structure with root directories 'trunk',
'branches' and 'tags', your source goes in trunk.
The program itself (sadly, I don't have the adapter to really test it):
- name the columns, not "column 0", "column 1", ...
- if you click "start" without having the adapter, the program dies with
a segmentation fault, that is not the way to handle errors :-)
Nice work for a start, anyway. Don't let me take your motivation :-)