Does Array.index need to be a foreign import?
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 4/5
- Tempo estimado
- 3-5 dias
- Facilidade para iniciantes
- 35/100
- Tipo de issue
- Refatoração
- Clareza
- Razoavelmente clara
- Status de atividade
- Estagnada
- Domínio
- backend
Direção de pesquisa
The issue names Array.index, length, and unsafeIndexImpl, and mentions the backend optimizer. Start by reading the implementation of Array.index and how its foreign import is used, then check whether the optimizer change described in the issue has landed. Done means determining whether the bounds check can be implemented in PureScript while preserving the expected behavior and enabling the stated optimization.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
The logic of Array.index can be implemented in terms of length and unsafeIndexImpl. Is it necessary for it to be foreign? In backend-optimizer, I'm preparing a change to inline the bounds check so that the Just allocation can be eliminated in some cases. If index bounds checking was implemented in PureScript rather than FFI, this would just work.
- Linguagem predominante
- PureScript
- Estrelas
- 58
- Forks
- 70
- Merge médio
- 2d 22h
- PRs com merge (30d)
- 1
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Tem um modelo de pull request
- Sem guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de purescript/purescript-arrays
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 55/100
purescript/purescript-arrays#246 ·
-
generate functionAberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 35/100
purescript/purescript-arrays#244 · 2 comentários ·
-
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 35/100
purescript/purescript-arrays#242 · 2 comentários ·
-
`groupBy` has unexpected `l` argument for `a -> a -> Boolean` predicateTalvez livre de novo Um pull request para esta issue foi fechado sem ser mesclado. Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 25/100
purescript/purescript-arrays#229 · 3 comentários ·
-
type: enhancement
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 30/100
purescript/purescript-arrays#215 · 3 comentários ·
Todas as issues de purescript/purescript-arrays
Issues semelhantes
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 62/100
-
ManageSieve proxy: a backend's NO/BYE greeting is skipped as a capability line and reported as EOFAbertabug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
Mantenedores costumam responder em até 1 dia
-
Mend: dependency security vulnerability
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 65/100
-
Mend: dependency security vulnerability
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 62/100
ManageIQ/manageiq-pods#1476 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 62/100
prime-radiant-inc/evener#6366 ·
Mantenedores costumam responder em até 1 dia