Hi Mauro,

First of all, there's no Linux DVB API v4. It was skipped, because there
was a proposal for a v4, with was never adopted.

Alright, whatever, you have understood it was the post-V3 API, S2API, you name it.
You should assign it a version number by the way.

Thanks for reviewing it! Yeah, the asterisks there are wrong.
The definitions should be, instead:

int ioctl(int fd, FE_SET_TONE, enum fe_sec_tone_mode tone)
int ioctl(int fd, FE_SET_VOLTAGE, enum fe_sec_voltage voltage)
int ioctl(int fd, FE_DISEQC_SEND_BURST, enum fe_sec_mini_cmd tone)

As they're passing by value, not by reference[1].

Thanks for the clarification.

Feel free to send us fix patches.

Do you suggest I should locate the repository, clone it, understand the structure, locate the documentation files, etc? That would take 20 times the time it takes to remove the 3 asterisk characters when you already master the source code as you
probably do.

I own a few opensource projects on sourceforge and github. When a user reports a problem, whether it is a functional one or a documentation typo, I fix it myself. I do not expect users to do it for me. For those projects, I am the developer and they are the users. I welcome contributions, but I do not demand or even expect them.

Cheers
-Thierry

Reply via email to