-
Notifications
You must be signed in to change notification settings - Fork 1
Adicionando 8.3 - Priority Queues #45
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
Conversation
|
@arademaker terminei. O único caveat é que fiz uma versão do deleteQ que retorna optional, porque pode ser o caso em que a gente precise aplicar um deleteQ numa lista nula pelo algoritmo do huffmanPQ |
arademaker
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
peço aplicar ajustes. Não esqueça de atualizar o README marcando a seção como 'feita'
|
@arademaker implementei as alterações pedidas |
arademaker
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
segue ultimo pedido para aceitar PR
|
@arademaker reverti para namespace em vez de section |
Estou abrindo o PR, mas ainda vou implementar mais código. Quando estiver pronto, vou marcar o @arademaker para revisar.