Added stdin reading capability to amqpsend #4

Merged
merged 1 commit into from Mar 2, 2012

Conversation

Projects
None yet
2 participants
Contributor

astephens25 commented Feb 27, 2012

I added the option to leave out the filename and the command line message and instead read the message body from standard input.

I performed regression testing to make sure the previous options still behaved generally as expected. The primary difference is where the help message was printed if the filename option and command-line message argument were missing.

Additional testing included sending of large files (20M) to test buffer reallocations. I did not do any performance analysis using the read buffer of only 8K.

Feel free to not accept these changes as you see fit. I made them for my own use and wish to give you the option to make them generally available in your project if you deem it desirable to do so.

Thank you.

rmt added a commit that referenced this pull request Mar 2, 2012

Merge pull request #4 from astephens25/amqpsend-stdin
Added stdin reading capability to amqpsend

@rmt rmt merged commit 29041cc into rmt:master Mar 2, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment