Specifying the library dependencies in the depends field of library.properties causes the Arduino Library Manager (Arduino IDE 1.8.10 and newer) to offer to install any missing dependencies during installation of this library. "arduino-cli lib install" will automatically install the dependencies (arduino-cli 0.7.0 and newer).
The _currentPageElement was not deleted on destruction. This caused a
memory leak. Furthermore there was an explicit destructor call which should
never be used. Now the memory handling is delegated to a unique_ptr which
makes it easier to track.
* This is more straight forward with C++11 and the way todo.
* Also move some initializations to the header file.
* Remove assignment of empty String("") since this is an assignment with
a default constructed object which is already default constructed.
* Remove reset of unique_ptr: They are per default initialized as empty nullptr.
Adding a custom deleter to unique_ptr in case of using the reference. This
makes the code less error prone and easier to read since we will only have
to use the creation of the pointer the correct way.
In some situations (not passing a WebServer instance from external) the
DNS server was not stopped in _stopPortal(). Maybe the intention was here not
to stopped an externally passed WebServer but I see no use-case for the
dns server here.
Fixed this to remove the enum handling in the next step.
It is not necessary to use a unique_ptr (also always using .release shows
this), so there can be used a raw pointer instead.
Here we fix usage of _aux an also of back pointing _ac. Especially on _ac
it does not make any sense, since this pointer is stored in many locations.
refs #138
Hieromon Ikasamo
per1234
Hieromon Ikasamo
Sven Steckmann