Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Better fix for "spectrogram.c: Fix an out-of-bounds access" #38

Closed
wants to merge 1 commit into from
Closed

Conversation

martinwguy
Copy link
Contributor

This reverts commit 91a7041,
which leaves a strip of black pixels at the top of the graph
visible only with --no-border, and makes a better fix for the same issue.

This reverts commit 91a7041,
which leaves a strip of black pixels at the top of the graph
visible only with --no-border, and makes a better fix for the same issue.
@erikd erikd closed this in 6ac4ecb Feb 27, 2016
@erikd
Copy link
Member

erikd commented Feb 27, 2016

Awesome. Thanks!

@martinwguy
Copy link
Contributor Author

Thanks to you!

BTW, you asked me some time ago for any ways you could be a more
effective OS project lead.
I can't say I have much experience or success in that field - I tend
to find myself developing my own bright ideas, maintaining other
people's abandonware or contributing little fixes to other projects -
but there is an excellent book on the subect, Producing Open Source
Software at producingoss.com which explores the area pretty well,
based on the experience of the Subversion project leads. Readable
online (but I bought a copy to read more easily). Whatever you may
think of SVN itself, his outline of the various mechanisms that are
used in OS development, and the interpersonal issues that arise, are
all valid.

Cheers

M

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants