MSVC warnings in environment.hpp and default_launcher.hpp
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 88/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- cpp
- Área
- operating-systems
Línea de trabajo
Empieza con los dos archivos mencionados en el informe: v2/environment.hpp cerca de la línea 1504, donde el bucle sobre pathext vuelve a declarar p y oculta una variable local externa, y v2/windows/default_launcher.hpp cerca de la línea 397, donde build_command_line_impl recibe un parámetro args que no se usa. Compila con MSVC para reproducir las advertencias C4456 y C4100. Está terminado cuando ambas advertencias desaparecen y las pruebas de Windows siguen pasando sin cambios de comportamiento.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
The first one is: "v2\environment.hpp(1504,18): warning C4456: declaration of 'p' hides previous local declaration"
In this code:
for (auto ext : pathext)
{
ec.clear();
BOOST_PROCESS_V2_NAMESPACE::filesystem::path nm(name);
nm.concat(ext.begin(), ext.end());
auto p = pp / nm;
if (detail::is_executable(p, ec) && !ec)
return p;
}
Should be probably changed to p = pp / nm;
The second one is on 2 lines (371, 397), showing only one as they are essentially the same: "v2\windows\default_launcher.hpp(397,39): warning C4100: 'args': unreferenced parameter"
In this code:
template<typename Argv>
static std::wstring build_command_line_impl(
const filesystem::path & pt,
const Argv & argv,
basic_string_view<char> args)
Commenting out the parameter should fix the warning.
- Lenguaje dominante
- C++
- Estrellas
- 145
- Forks
- 151
- 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 boostorg/process
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 72/100
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 68/100
-
v2: BOOST_PROCESS_V2_DISABLE_SIGNALSET does not compile with the signal-based process handlePosiblemente ocupada @kislinsk la tomó hace 3 días. Abierto
Dificultad 3/5 Medio día Aptitud para principiantes 62/100
-
Why does it synchronously wait?Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
Todos los issues de boostorg/process
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
nasa/fprime#6225 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
[Feature]: Windows 语音输入默认关闭Abiertoenhancement source/qq
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
metasequoiaime/msime-windows#1054 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
amnezia-vpn/amnezia-client#3297 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
[安卓Feature]: 数字键盘左侧符号区域大小优化Abiertoenhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
metasequoiaime/msime#8620 ·
Los mantenedores suelen responder en 1 día
-
area/docs type/bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
ChiR24/Unreal_mcp#679 ·
Los mantenedores suelen responder en 1 día