microsoft / microsoft/libHttpClient

XTaskQueueSetCurrentProcessTaskQueue SAL should use _In_opt_.

Offen Anfängerfreundlich
#958 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

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

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Ö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

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.