Add new options to MISRC Capture

[alvin8t]

Added new options:

• A suppresses all clipping messages from ADC A
• B suppresses all clipping messages from ADC B
• t takes the integer entered and multiples it by 40,000,000 (for entering number of seconds)
• w overwrites already existing files without asking
• Fixed n where it did not work at all, but now will exit when the number of samples is reached. This takes priority over -t (if specified at the same time).

First time committing to an open source project, any advice or suggestions welcome, and please excuse my ignorance on my mistakes.

[harrypm]

Just did a quick test, this appears to be working well.

(Ideally also need to add threads option to flac now with the v1.5.0 multithreading update)

[Stefan-Olt]

Thanks for the pull request! This is first open source project I'm maintaining, so all I can tell you is that I see nothing wrong with the way you created this pull request.
I'll have a look at your code and comment there if I notice something

[Stefan-Olt]

I wrote comments directly in the code

[Stefan-Olt]

Not sure if -t is the best option, maybe that's better for FLAC threads. Why not -s?

[Stefan-Olt]

Harry suggested that this could be extended into hh:mm:ss (a single value is treated as seconds, if there is a single colon it's mm:ss, with two colons it's hh:mm:ss). Then -s doesn't make much sense

[alvin8t]

That sounds like a better method. I initially choose t for time. I thought about doing it where the user could enter "4h5m34s", but I am not well versed in the C language and didn't want to learn to deal with pointers, memory, etc. The languages I primarily work in are PHP, Java, and RPGLE, so it's hard for me to create the parsing for that in C.

[Stefan-Olt]

Most things are harder in C, but I could add that at a later point

[Stefan-Olt]

Could you fix the brackets and update your code to work with the flac threading (the command parsing string needs to be combined)? Then I would merge it and later add the hh:mm:ss parsing