microsoft / microsoft/libHttpClient
XTaskQueueSetCurrentProcessTaskQueue SAL should use _In_opt_.
Abierto
Apto para principiantes
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- C++
- Estrellas
- 295
- Forks
- 144
- Merge medio
- 2 d 19 h
- PR fusionados (30 d)
- 2
Descripción
The SAL for XTaskQueueSetCurrentProcessTaskQueue is incorrect; it should be allowing a null parameter.
/// <summary>
/// Sets the given task queue as the process wide task queue. The
/// queue can be set to nullptr, in which case XTaskQueueGetCurrentProcessTaskQueue will
/// also return nullptr. The provided queue will have its handle duplicated
/// and any existing process task queue will have its handle closed.
/// </summary>
/// <param name='queue'>The queue to set up as the default task queue for the procces.</param>
STDAPI_(void) XTaskQueueSetCurrentProcessTaskQueue(
_In_ XTaskQueueHandle queue
) noexcept;
That should use _In_opt_.
Guía de contribución
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.
Línea de trabajo
Comienza con la declaración de XTaskQueueSetCurrentProcessTaskQueue mostrada en el issue e inspecciona las declaraciones de la API pública circundantes. Actualiza la anotación del parámetro de la cola para permitir nullptr y, a continuación, verifica que la declaración coincida con el comportamiento documentado y con cualquier comprobación relacionada de la API.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- cpp
- Área
- api
- Tipo de issue
- Error
- Dificultad
- 1/5
- Tiempo estimado
- Menos de una hora
- Estado de actividad
- Tranquilo
- Claridad
- Bien especificado
- Aptitud para principiantes
- 65/100