microsoft / microsoft/libHttpClient
XTaskQueueSetCurrentProcessTaskQueue SAL should use _In_opt_.
Offen
Anfängerfreundlich
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- C++
- Sterne
- 295
- Forks
- 144
- Ø Merge
- 2 T. 19 Std.
- Gemergte PRs (30 T.)
- 2
Beschreibung
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_.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginnen Sie mit der im Issue gezeigten Deklaration von XTaskQueueSetCurrentProcessTaskQueue und prüfen Sie die umgebenden Deklarationen der öffentlichen API. Aktualisieren Sie die Annotation des Queue-Parameters, sodass nullptr zulässig ist, und überprüfen Sie anschließend, dass die Deklaration dem dokumentierten Verhalten und allen zugehörigen API-Prüfungen entspricht.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- cpp
- Bereich
- api
- Issue-Typ
- Bug
- Schwierigkeit
- 1/5
- Geschätzter Aufwand
- Unter einer Stunde
- Aktivitätsstatus
- Ruhig
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 65/100