set default value for '_onDisconnectfn'

[lumapu]

The value is checked here to be different from zero:

ESPAsyncWebServer/src/WebRequest.cpp

Lines 266 to 268 in 2803989

	if (_onDisconnectfn) {
	_onDisconnectfn();
	}

To have a defined behavior I would recommend to set _onDisconnectfn to NULL

[Copilot]

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

[Copilot]

Pull Request Overview

This PR initializes the newly added _onDisconnectfn member to NULL in the AsyncWebServerRequest constructor, preventing undefined behavior when the pointer is later checked.

• Added _onDisconnectfn(NULL) to the initializer list.

Comments suppressed due to low confidence (2)

src/WebRequest.cpp:25

• [nitpick] Consider using nullptr instead of NULL for pointer initialization in C++ to improve type safety and clarity.

  : _client(c), _server(s), _handler(NULL), _response(NULL), _onDisconnectfn(NULL), _temp(), _parseState(PARSE_REQ_START), _version(0), _method(HTTP_ANY),

src/WebRequest.cpp:25

• [nitpick] Ensure the initializer list order matches the declaration order of class members to avoid compiler warnings and improve maintainability.

  : _client(c), _server(s), _handler(NULL), _response(NULL), _onDisconnectfn(NULL), _temp(), _parseState(PARSE_REQ_START), _version(0), _method(HTTP_ANY),