Skip to content
New issue

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

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

Already on GitHub? # to your account

Sobrecarga para operador () #11

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

lordcobisco
Copy link

Inserindo um intervalo, é possível obter um conjunto de linhas e colunas
predefinidas em um intervalo inferior e superior.

Inserindo um intervalo, é possível obter um conjunto de linhas e colunas
predefinidas em um intervalo inferior e superior.
@rodrigoavalente
Copy link
Contributor

Tem umas coisas que eu gostaria que o senhor alterasse antes de realizar esse pull.

  • Escrevi um material sucinto para você entender melhor o que eu quis dizer com a classe ser interna a matriz. Link do Gist
  • Eu reparei que o .h possui chamadas diferentes do .hpp, isso irá causar erro de qualquer forma, entretanto, o .hpp possui definições melhores dos parâmetros do que o .h, evite isso, o que está escrito no .h é o que vai aparecer ao usuário quando utilizar a classe.
  • Conforme a conversa no fb, eu gostaria que o senhor abrisse uma issue mais detalhada com o problema, o motivo disso é para mantermos uma referência e deixar documentado no que estamos trabalhando para consultas futuras, é pertinente deixar na issue materiais que exemplificam seu ponto de vista, como links de conteúdos da internet, seria interessante que o senhor usasse markdown para deixar códigos exemplos de qual é a sua ideia, de forma a deixar mais claro, por isso vou deixar o link da documentação de markdown do github aqui, não se preocupe md é besteira coisa que se aprende em 5min.
  • Também reparei que você enviou seu código de sua branch master. Isso é ruim, você deve manter a branch master limpa e somente com atualizações do repositório oficial, pois em casa de problemas com uma nova feature e tendo que fazer o checkout, é possível que você perca trabalho que não gostaria.

Vo tentar realizar as alterações q vc pediu no sistema de controle.
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants