constructor accepting an universal reference may interfere with move constructor

add const qualification, remove explicit initialization of string to empty, and
unify parameter names between declaration and definition

primitive types don't need std::move

Tomáš Pecka authored 4 years ago and Jan Trávníček committed 4 years ago

Although #pragma once is not a standard, compilers nowadays usually
support it. We don't have any tricky build system so we should be OK.

This also fixes many errors regarding using reserved identifiers (names
starting with double underscore).

removal of an unnecessary recursion, addition of forgotten move, removal of
unnecessary const qualifier, change int to -> size_t

Remove CMakeLists.txt generator scripts from build phase. Replaced with
CMake's GLOB that can re-run every time the build is invoked. We can
also drop the Python dependency now.

Removed header installs which will be redesigned in future commits.

Tomáš Pecka authored 4 years ago and Tomáš Pecka committed 4 years ago

Remove Dot2 algorithm. By default, the interactive window now
does not block CLI. Added overload to override this behaviour.

Closes #199