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

faidx with long opts + output filename + line length #509

Closed
wants to merge 1 commit into from

Conversation

lindenb
Copy link
Contributor

@lindenb lindenb commented Jan 12, 2016

some Minor changes in faidx:

  • added support for long_opt
  • added option -o|--output (default:stdout)
  • added option -n|--length to specify the length of the lines in the fasta output

@whitwham
Copy link
Contributor

Rebased and merged via 1e23e1c

@whitwham whitwham closed this Apr 18, 2018
@tseemann
Copy link

tseemann commented Apr 21, 2018

@lindenb thanks for this! fadix needed some "TLC".

I am surprised that putchar() was used. I guess it's a macro/inline and uses stdio buffering so shouldn't be too inefficient, and makes the code simple.

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

Successfully merging this pull request may close these issues.

5 participants