Avoid reflect.MethodByName
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 45/100
Línea de trabajo
Empieza por rastrear cómo expr.Function() declara funciones y dónde se usa reflect.Method() o reflect.MethodByName(). Define el comportamiento de exclusión para una etiqueta de estilo exprdeclarative, incluido el impacto en el builtin get, y verifica que la reflexión no constante ya no impida la eliminación de código muerto.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Hey!
When using reflect.Method() or reflect.MethodByName() with a non-constant argument, dead code elimination is disabled for public functions. This makes the final binary bigger than it should. See for example https://github.com/kubernetes/kubernetes/issues/132216 for an effort on the subject.
I am using the expr.Function() to declare function, so I think it should be possible to avoid calling reflect.MethodByName(). Would you consider a tag "exprdeclarative" or something like that to opt out reflect?
This would also disable some builtins, like get.
- Lenguaje dominante
- Go
- Estrellas
- 8k
- Forks
- 535
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de expr-lang/expr
-
docs needed
Dificultad 1/5 1-3 horas Aptitud para principiantes 62/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 65/100
-
`find`, `findIndex`, `first` and `get` are type-checked as the element type where they return nilAbierto
Dificultad 3/5 1-2 días Aptitud para principiantes 65/100
-
Release latest master branchAbierto
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
Todos los issues de expr-lang/expr
Issues similares
-
Broken links in the docsAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día
-
type/bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
[E2E Scenario Tests] HTTP logs capture export requests from test framework, polluting golden filesAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
GoogleCloudPlatform/k8s-config-connector#13675 ·
Los mantenedores suelen responder en 1 día
-
ai-inspected
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
Los mantenedores suelen responder en 1 día
-
Usage of `fallthrough` keyword.Abiertotype/bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día