microsoft / microsoft/libHttpClient
XTaskQueueSetCurrentProcessTaskQueue SAL should use _In_opt_.
Aperta
Adatta ai principianti
Nessuno ha ancora preso questa issue.
- Lingua principale
- C++
- Stelle
- 295
- Fork
- 144
- Merge medio
- 2g 19h
- PR unite (30g)
- 2
Descrizione
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_.
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia dalla dichiarazione di XTaskQueueSetCurrentProcessTaskQueue mostrata nell’issue e ispeziona le dichiarazioni circostanti dell’API pubblica. Aggiorna l’annotazione del parametro della coda per consentire nullptr, quindi verifica che la dichiarazione corrisponda al comportamento documentato e a tutti i controlli API correlati.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- cpp
- Ambito
- api
- Tipo di issue
- Bug
- Difficoltà
- 1/5
- Tempo stimato
- Meno di un'ora
- Stato di attività
- Tranquilla
- Chiarezza
- Specificata chiaramente
- Idoneità per principianti
- 65/100