Skip to content

Sobrecarga para operador ()#11

Open
lordcobisco wants to merge 2 commits intoEngineeringLibrary:masterfrom
lordcobisco:master
Open

Sobrecarga para operador ()#11
lordcobisco wants to merge 2 commits intoEngineeringLibrary:masterfrom
lordcobisco:master

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.
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.

2 participants

Comments