Replace Rollback with Scope

[zhiltsov-max]

Summary

Replaces error rollback utilities with scope utilities. The new class extends the previous situation to be like try-except-finally and allows to enter context managers. This helps to maintain flat code indentation and helps with the objects that require closing / disposal in any cases (i.e. resource managers). The Project class is going to be such a manager - this is needed to resolve problems with file removal on Windows.

How to test

Checklist

• I submit my changes into the develop branch
• I have added description of my changes into CHANGELOG
• I have updated the documentation accordingly
• I have added tests to cover my changes
• I have linked related issues

License

• I submit my code changes under the same MIT License that covers the project.
  Feel free to contact the maintainers if that's a concern.
• I have updated the license header for each file (see an example below)

# Copyright (C) 2021 Intel Corporation
#
# SPDX-License-Identifier: MIT

[zhiltsov-max]

A draft for automatic cm closing:

def test_cm_closing(self):
        import inspect
        this_f = inspect.currentframe()
        begin_values = set(id(v) for v in this_f.f_locals.values())

        f2 = inspect.getframeinfo(this_f)
        q = 5

        end_values = set(id(v) for v in this_f.f_locals.values())
        ids_to_release = end_values - begin_values
        for v in this_f.f_locals.values():
            if id(v) in ids_to_release:
                if hasattr(v, 'close'):
                    v.close()
                elif hasattr(v, '__exit__'):
                    v.__exit__()

Unfortunately, it looks extremely fragile and restricted.

[zhiltsov-max]

@IRDonch, I refactored and simplified the class.

[IRDonch]

Suggested change

	ignore_errors: bool = False):
	ignore_errors: bool = False) -> None:

(and ditto for the other None-returning functions)

[zhiltsov-max]

What is the point in this?

[IRDonch]

To help typecheckers find errors.

[zhiltsov-max]

So you suggest to add it to all functions without return value?

[IRDonch]

I would say yes (if there are annotations on the function's arguments). Otherwise, the return type defaults to Any.

[zhiltsov-max]

I'm not convinced it is worthy investment of efforts in most cases. I'm open for a PR on this, but I don't think it is needed for the functions, which names don't not suppose any return value.