upstream/mercurial-mirror Files · mercurial/thirdparty/attr/_config.py

transaction: make entries a private attribute (API)...

transaction: make entries a private attribute (API) This attribute is tracking changes to append-only files. It is an implementation detail and should not be exposed as part of the public interface. But code in repair was accessing it, so it seemingly does belong as part of the public API. But that code in repair is making assumptions about how storage works and is grossly wrong when alternate storage backends are in play. We'll need some kind of "strip" API at the storage layer that knows how to handle things in a storage-agnostic manner. I don't think accessing a private attribute on the transaction is any worse than what this code is already doing. So I'm fine with violating the abstraction for transactions. And with this change, all per-instance attributes on transaction have been made private except for "changes" and "hookargs." Both are used by multiple consumers and look like they need to be part of the public interface. .. api:: Various attributes of ``transaction.transaction`` are now ``_`` prefixed to indicate they shouldn't be used by external consumers. Differential Revision: https://phab.mercurial-scm.org/D4634

Siddharth Agarwal - - Load All Authors

File last commit:

r34398:765eb17a default


                r39722:3d22aef3

default

Download file

             _config.py
        
                    23 lines
            
             | 514 B
            
                | text/x-python
            
             |
                PythonLexer
            
             / mercurial / thirdparty / attr / _config.py
          
                    History
                
                 |
                  Annotation
                 | Raw
                 |Copy content
                 |Copy permalink

      from __future__ import absolute_import, division, print_function

      __all__ = ["set_run_validators", "get_run_validators"]

      _run_validators = True

      def set_run_validators(run):

          """

          Set whether or not validators are run.  By default, they are run.

          """

          if not isinstance(run, bool):

              raise TypeError("'run' must be bool.")

          global _run_validators

          _run_validators = run

      def get_run_validators():

          """

          Return whether or not validators are run.

          """

          return _run_validators

	Site-wide shortcuts
/	Use quick search box
g h	Goto home page
g g	Goto my private gists page
g G	Goto my public gists page
g 0-9	Goto bookmarked items from 0-9
n r	New repository page
n g	New gist page

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository access permissions settings
t s	Toggle sidebar on some pages

				from __future__ import absolute_import, division, print_function


				__all__ = ["set_run_validators", "get_run_validators"]

				_run_validators = True


				def set_run_validators(run):
				"""
				Set whether or not validators are run. By default, they are run.
				"""
				if not isinstance(run, bool):
				raise TypeError("'run' must be bool.")
				global _run_validators
				_run_validators = run


				def get_run_validators():
				"""
				Return whether or not validators are run.
				"""
				return _run_validators