Sample

pyint-7878b5ed7b31-I10

Deterministically remove some optional blank lines and bracketed line breaks

Matched original source

Lines269

Scores

MethodScoreScale
Posnett00–1; higher is more readable
Dorn0.6070–1; higher is more readable
Mi0.4290–1; higher is more readable
LLOC145Logical lines; fewer is more readable
DeepSeek V4 Pro10.670–20; mean of three runs
GPT-6.1 Sol14.330–20; mean of three runs
BTTF0.560–1; higher is more readable

Views

Source code

from __future__ import annotations import calendar import copy import time from collections.abc import Iterator, MutableMapping from http.cookiejar import Cookie, CookieJar, CookiePolicy from typing import TYPE_CHECKING, Any, TypeVar, overload from ._internal_utils import to_native_string from ._types import is_prepared as _is_prepared from .compat import Morsel, cookielib, urlparse, urlunparse import threading class RequestsCookieJar(CookieJar, MutableMapping[str, str | None]): # type: ignore[misc] """Compatibility class; is a http.cookiejar.CookieJar, but exposes a dict interface. This is the CookieJar we create by default for requests and sessions that don't specify one, since some clients may expect response.cookies and session.cookies to support dict operations. Requests does not use the dict interface internally; it's just for compatibility with external client code. All requests code should work out of the box with externally provided instances of ``CookieJar``, e.g. ``LWPCookieJar`` and ``FileCookieJar``. Unlike a regular CookieJar, this class is pickleable. .. warning:: dictionary operations that are normally O(1) may be O(n). """ _policy: CookiePolicy def get( # type: ignore[override] self, name: str, default: str | None = None, domain: str | None = None, path: str | None = None, ) -> str | None: """Dict-like get() that also supports optional domain and path args in order to resolve naming collisions from using one cookie jar over multiple domains. .. warning:: operation is O(n), not O(1). """ try: return self._find_no_duplicates(name, domain, path) except KeyError: return default def set( self, name: str, value: str | Morsel[dict[str, str]] | None, **kwargs: Any ) -> Cookie | None: """Dict-like set() that also supports optional domain and path args in order to resolve naming collisions from using one cookie jar over multiple domains. """ # support client code that unsets cookies by assignment of a None value: if value is None: remove_cookie_by_name( self, name, domain=kwargs.get("domain"), path=kwargs.get("path") ) return if isinstance(value, Morsel): c = morsel_to_cookie(value) else: c = create_cookie(name, value, **kwargs) self.set_cookie(c) return c def iterkeys(self) -> Iterator[str]: """Dict-like iterkeys() that returns an iterator of names of cookies from the jar. .. seealso:: itervalues() and iteritems(). """ for cookie in iter(self): yield cookie.name def keys(self) -> list[str]: # type: ignore[override] """Dict-like keys() that returns a list of names of cookies from the jar. .. seealso:: values() and items(). """ return list(self.iterkeys()) def itervalues(self) -> Iterator[str | None]: """Dict-like itervalues() that returns an iterator of values of cookies from the jar. .. seealso:: iterkeys() and iteritems(). """ for cookie in iter(self): yield cookie.value def values(self) -> list[str | None]: # type: ignore[override] """Dict-like values() that returns a list of values of cookies from the jar. .. seealso:: keys() and items(). """ return list(self.itervalues()) def iteritems(self) -> Iterator[tuple[str, str | None]]: """Dict-like iteritems() that returns an iterator of name-value tuples from the jar. .. seealso:: iterkeys() and itervalues(). """ for cookie in iter(self): yield cookie.name, cookie.value def items(self) -> list[tuple[str, str | None]]: # type: ignore[override] """Dict-like items() that returns a list of name-value tuples from the jar. Allows client-code to call ``dict(RequestsCookieJar)`` and get a vanilla python dict of key value pairs. .. seealso:: keys() and values(). """ return list(self.iteritems()) def list_domains(self) -> list[str]: """Utility method to list all the domains in the jar.""" domains: list[str] = [] for cookie in iter(self): if cookie.domain not in domains: domains.append(cookie.domain) return domains def list_paths(self) -> list[str]: """Utility method to list all the paths in the jar.""" paths: list[str] = [] for cookie in iter(self): if cookie.path not in paths: paths.append(cookie.path) return paths def multiple_domains(self) -> bool: """Returns True if there are multiple domains in the jar. Returns False otherwise. :rtype: bool """ domains: list[str] = [] for cookie in iter(self): if cookie.domain is not None and cookie.domain in domains: # type: ignore[reportUnnecessaryComparison] # defensive check return True domains.append(cookie.domain) return False # there is only one domain in jar def get_dict( self, domain: str | None = None, path: str | None = None ) -> dict[str, str | None]: """Takes as an argument an optional domain and path and returns a plain old Python dict of name-value pairs of cookies that meet the requirements. :rtype: dict """ dictionary: dict[str, str | None] = {} for cookie in iter(self): if (domain is None or cookie.domain == domain) and ( path is None or cookie.path == path ): dictionary[cookie.name] = cookie.value return dictionary def __iter__(self) -> Iterator[Cookie]: # type: ignore[override] """RequestCookieJar's __iter__ comes from CookieJar not MutableMapping.""" return super().__iter__() def __contains__(self, name: object) -> bool: try: return super().__contains__(name) except CookieConflictError: return True def __getitem__(self, name: str) -> str | None: """Dict-like __getitem__() for compatibility with client code. Throws exception if there are more than one cookie with name. In that case, use the more explicit get() method instead. .. warning:: operation is O(n), not O(1). """ return self._find_no_duplicates(name) def __setitem__( self, name: str, value: str | Morsel[dict[str, str]] | None ) -> None: """Dict-like __setitem__ for compatibility with client code. Throws exception if there is already a cookie of that name in the jar. In that case, use the more explicit set() method instead. """ self.set(name, value) def __delitem__(self, name: str) -> None: """Deletes a cookie given a name. Wraps ``http.cookiejar.CookieJar``'s ``remove_cookie_by_name()``. """ remove_cookie_by_name(self, name) def set_cookie(self, cookie: Cookie, *args: Any, **kwargs: Any) -> None: if ( (value := cookie.value) is not None and value.startswith('"') and value.endswith('"') ): cookie.value = value.replace('\\"', "") return super().set_cookie(cookie, *args, **kwargs) def update( # type: ignore[override] self, other: CookieJar | SupportsKeysAndGetItem[str, str] ) -> None: """Updates this jar with cookies from another CookieJar or dict-like""" if isinstance(other, cookielib.CookieJar): for cookie in other: self.set_cookie(copy.copy(cookie)) else: super().update(other) def _find( self, name: str, domain: str | None = None, path: str | None = None ) -> str | None: """Requests uses this method internally to get cookie values. If there are conflicting cookies, _find arbitrarily chooses one. See _find_no_duplicates if you want an exception thrown if there are conflicting cookies. :param name: a string containing name of cookie :param domain: (optional) string containing domain of cookie :param path: (optional) string containing path of cookie :return: cookie.value """ for cookie in iter(self): if cookie.name == name: if domain is None or cookie.domain == domain: if path is None or cookie.path == path: return cookie.value raise KeyError(f"name={name!r}, domain={domain!r}, path={path!r}") def _find_no_duplicates( self, name: str, domain: str | None = None, path: str | None = None ) -> str: """Both ``__get_item__`` and ``get`` call this function: it's never used elsewhere in Requests. :param name: a string containing name of cookie :param domain: (optional) string containing domain of cookie :param path: (optional) string containing path of cookie :raises KeyError: if cookie is not found :raises CookieConflictError: if there are multiple cookies that match name and optionally domain and path :return: cookie.value """ toReturn = None for cookie in iter(self): if cookie.name == name: if domain is None or cookie.domain == domain: if path is None or cookie.path == path: if toReturn is not None: # if there are multiple cookies that meet passed in criteria raise CookieConflictError( f"There are multiple cookies with name, {name!r}" ) # we will eventually return this as long as no cookie conflict toReturn = cookie.value if toReturn is not None: return toReturn raise KeyError(f"name={name!r}, domain={domain!r}, path={path!r}") def __getstate__(self) -> dict[str, Any]: """Unlike a normal CookieJar, this class is pickleable.""" state = self.__dict__.copy() # remove the unpickleable RLock object state.pop("_cookies_lock") return state def __setstate__(self, state: dict[str, Any]) -> None: """Unlike a normal CookieJar, this class is pickleable.""" self.__dict__.update(state) if "_cookies_lock" not in self.__dict__: self._cookies_lock = threading.RLock() def copy(self) -> RequestsCookieJar: """Return a copy of this RequestsCookieJar.""" new_cj = RequestsCookieJar() new_cj.set_policy(self.get_policy()) new_cj.update(self) return new_cj def get_policy(self) -> CookiePolicy: """Return the CookiePolicy instance used.""" return self._policy

Loading feature diagnosis…

Source regions

Feature contributions

Feature contributions

FeatureValueContribution
(log-odds)
Halstead volumeHow much symbolic information the code contains. It combines the total number of operators and values with the number of distinct ones.8743.824−288.546
Source linesThe number of non-empty source lines.269+107.6
Byte entropyHow varied the bytes in the source text are. Repeated bytes give a lower value; a more even mixture gives a higher value.4.379−6.569

Intercept 8.87 + feature contributions = -178.645 (log-odds).
Score = sigmoid(-178.645) = 0.

Feature contributions

FeatureValueContribution
(log-odds)
Comment layout frequencyHow frequently the layout switches between lines containing only comments or whitespace and lines containing code.269−8.488
Indentation frequencyHow rapidly indentation changes from line to line.175+5.316
Identifier layout (vertical)How rapidly the visual pattern of identifier characters changes from top to bottom.268.837+3.348
Identifier layout (horizontal)How rapidly the visual pattern of identifier characters changes from left to right. It measures spatial frequency, not the meaning of the names.133.974−2.009
Comment layout (horizontal)How rapidly the visual pattern of comment characters changes from left to right.133.978+1.357
Operator visual areaThe share of the code’s visual area occupied by operators such as +, =, and *.0.051+0.443
Number layout frequencyHow rapidly the number of numeric values per line changes through the code.1+0.398

Intercept 0.07 + feature contributions = 0.435 (log-odds).
Score = sigmoid(0.435) = 0.607.

Score0.429
Score145
Run 1

The code is generally readable and well-structured, but has some inconsistencies that detract from clarity. Class methods are logically organized and docstrings are helpful. However, there are several formatting issues: inconsistent spacing in function signatures (e.g., `set_cookie`, `_find_no_duplicates`), missing blank lines between some methods, and misaligned closing braces in chained method calls. The `_find_no_duplicates` method has a nested conditional structure that could be flattened for clarity, and variable naming like `toReturn` breaks Python conventions. Some methods like `multiple_domains` and `list_domains` have redundant logic. While the overall purpose is clear and the code is concise, these inconsistencies in action granularity and reading flow prevent it from scoring higher.

Run 2

The code suffers from severe readability issues. Structure is inconsistent: methods are packed together with little or no spacing, while some signatures are broken awkwardly across lines with odd indentation and parameter alignment. Nesting is excessive in methods like _find and _find_no_duplicates, creating deep conditional chains. Clarity of intent is undermined by inconsistent naming (toReturn instead of snake_case), misplaced warnings, and unclear comments. Action granularity is poor: many lines contain multiple statements or bizarrely wrapped expressions. Reading flow is disjointed due to erratic line breaks, unexpected placement of return type annotations (e.g., '-> str' on separate line), and inconsistent formatting. While some docstrings are helpful, they are often poorly formatted and not enough to offset the overall lack of polish. The code is functional but not readable.

Run 3

The code has a clear class structure and useful docstrings, but readability suffers from inconsistent spacing, misaligned function parameters, and inconsistent blank lines between methods. Several method signatures have awkward line breaks (e.g., `get()` and `set()`), and some closing parentheses are placed far from their function calls. The `get_dict()` method has overly long lines and multi-line conditions that could be simplified. Nested conditionals in `_find_no_duplicates()` create deep nesting (4 levels), making the logic harder to follow. While the core intent is understandable, the formatting is disorganized and detracts from the reading flow. Overall, the code is functional but would benefit from consistent formatting, reduced nesting, and cleaner parameter alignment.

Run 1

Code Structure: Methods are logically separated into mapping operations, cookie lookup, and serialization, though inconsistent spacing weakens organization. Nesting: Most methods are shallow, but lookup helpers contain three to four nested conditionals that could be flattened. Clarity of intent: Descriptive method names, type annotations, and extensive docstrings explain the compatibility interface well; terse variables such as c and toReturn, plus some misleading documentation, reduce clarity. Code length: Most methods are concise, but repetitive wrappers, verbose docstrings, and duplicated filtering logic add bulk. Action granularity: Statements generally perform one action per line; the assignment expression in set_cookie slightly increases cognitive load. Reading flow: Related methods mostly follow a natural sequence, but missing blank lines, irregular signature formatting, excessive inline whitespace, and long comments interrupt scanning.

Run 2

Code Structure: Cohesive class with small, logically related methods, though inconsistent spacing between methods weakens organization. Nesting: Most methods are shallow, but lookup helpers use three to four nested conditions that could be flattened. Clarity of intent: Descriptive method names, type annotations, and docstrings explain the interface well; vague variables such as c and toReturn, plus misleading documentation in multiple_domains and __setitem__, reduce clarity. Code length: Individual methods are generally concise, but repetitive wrappers and extensive boilerplate docstrings make the class lengthy. Action granularity: Statements generally perform one action per line; the assignment expression in set_cookie slightly increases cognitive load. Reading flow: Related methods are grouped naturally, but irregular signature formatting, excessive inline spacing, long comments, and missing blank lines interrupt scanning.

Run 3

Code Structure: Cohesive class with small, logically related methods, though inconsistent spacing and import organization weaken presentation. Nesting: Most methods are shallow, but the cookie lookup helpers contain three to four nested conditions that could be flattened. Clarity of intent: Descriptive method names, type annotations, and extensive docstrings explain the interface well; generic names such as c and toReturn, plus occasional mismatches between documentation and implementation, reduce clarity. Code length: Individual methods are concise, but repetitive iteration patterns and verbose boilerplate documentation make the class longer than necessary. Action granularity: Almost every statement performs one action; the assignment expression in set_cookie slightly increases parsing effort. Reading flow: Related operations generally appear together, but missing blank lines, awkward signature wrapping, excessive inline whitespace, and long suppression comments interrupt otherwise natural reading.