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

Already on GitHub? Sign in to your account

bam2bcf.c ln 57 seems redundant. #12

Closed
xied75 opened this Issue Jun 7, 2012 · 3 comments

Comments

Projects
None yet
2 participants

xied75 commented Jun 7, 2012

memset(r, 0, sizeof(bcf_callret1_t));
Collaborator

pd3 commented Jun 7, 2012

It's not, why do you think so? See bam_tview.c. Even if it was, it certainly is not a bottleneck.

@pd3 pd3 closed this Jun 7, 2012

xied75 commented Jun 7, 2012

Because ln 44 already done that once? I didn't say it's a bottleneck, I just saw that when I'm surfing the code.

Collaborator

pd3 commented Jun 7, 2012

Apologies, I should have looked more carefully. You are right, the line is redundant.

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