search

rblatss / Linux-Wav-Player-Suite

An exercise in C++ to play/plot .wav files from BASH. Written in the spirit of the unix hallmark: Write programs that do one thing, and do it well.
0 stars 0 forks source link

Code review #1

Open rblatss opened 9 months ago

rblatss commented 9 months ago

I wrote this a while ago and I don't think it totally works. I made some naive design choices. It sounds kind of fun to tear up and re-stitch what I did. And since creating this project, I've done some development in C++ for Linux at work (and a little embedded C too). I'd like to get an idea of what I've learned and where I'm at with Linux / C++ development.

I'd like to do this as a GitHub code review. Unfortunately, I never planned on doing reviews in this repo; all of my commits are to master. Need to figure out my options.

rblatss commented 9 months ago

Well, since I already have a couple things in mind that I want to fix, I'll just open a pull request with the branch linked to this issue (it's called code-review) for these things. Then, I can do a further review with comments via that request.