Daveisfera pr 397

[sirknightj]

#394 / #397, with some small modifications + running the tests

1. Use IS_EMPTY_STRING macro from PIC
2. Use the maximum lengths defined in https://docs.aws.amazon.com/kinesisvideostreams/latest/dg/limits.html#limits-streaming-metadata instead of arbitrary 200 (which goes past 128 for key length).
3. Use SNPRINTF macro from PIC instead of sprintf.
4. Update the README.

Manual tests:

• ./kvsVideoOnlyRealtimeStreamingSample test-stream h264 "" ../samples 0 -- works ✅

• ./kvsVideoOnlyRealtimeStreamingSample test-stream h264 "" ../samples 11 -- errors and exits ✅

• ./kvsVideoOnlyRealtimeStreamingSample test-stream h264 "" ../samples 9 -- works ✅

• See the mkv tags appended using mkvinfo. Procedure:

export KVS_DEBUG_DUMP_DATA_FILE_DIR=`pwd`/media
mkdir media
./kvsVideoOnlyRealtimeStreamingSample test-stream
mkvinfo -v ./media/test-stream_0.mkv

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

[jdelapla]

Add instructions to README.md for new arguments.

[codecov-commenter]

Codecov Report

Attention: 18 lines in your changes are missing coverage. Please review.

Comparison is base (4f1e798) 62.83% compared to head (ef275ac) 62.76%.

❗ Current head ef275ac differs from pull request most recent head 27d593b. Consider uploading reports for the commit 27d593b to get more accurate results

Files	Patch %	Lines
samples/KvsVideoOnlyRealtimeStreamingSample.c	0.00%	18 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop     #421      +/-   ##
===========================================
- Coverage    62.83%   62.76%   -0.08%     
===========================================
  Files           32       32              
  Lines         4599     4611      +12     
===========================================
+ Hits          2890     2894       +4     
- Misses        1709     1717       +8     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.