From e0120c08672009f8d4445eebef8efb22ddae5fb3 Mon Sep 17 00:00:00 2001 From: Benedikt Peetz Date: Tue, 17 Jun 2025 08:59:14 +0200 Subject: refactor(yt_dlp/logging): Avoid adding to the `__all__` list This seems to have no apparent effect on anything. As such I went ahead and removed this dead code. --- crates/yt_dlp/src/logging.rs | 28 +--------------------------- 1 file changed, 1 insertion(+), 27 deletions(-) (limited to 'crates/yt_dlp/src') diff --git a/crates/yt_dlp/src/logging.rs b/crates/yt_dlp/src/logging.rs index 5cb4c1d..112836e 100644 --- a/crates/yt_dlp/src/logging.rs +++ b/crates/yt_dlp/src/logging.rs @@ -15,7 +15,7 @@ use log::{Level, MetadataBuilder, Record, logger}; use rustpython::vm::{ PyObjectRef, PyRef, PyResult, VirtualMachine, - builtins::{PyInt, PyList, PyStr}, + builtins::{PyInt, PyStr}, convert::ToPyObject, function::FuncArgs, }; @@ -167,31 +167,5 @@ def basicConfig(*pargs, **kwargs): "".to_owned(), )?; - let all: PyRef = logging - .get_attr("__all__", vm)? - .downcast() - .expect("Is a list"); - all.borrow_vec_mut().push(vm.new_pyobj("HostHandler")); - - // { - // let logging_dict = logging.dict().expect("Exists"); - // - // for (key, val) in scope.globals { - // let key: PyRef = key.downcast().expect("Is a string"); - // - // if !logging_dict.contains_key(key.as_str(), vm) { - // logging_dict.set_item(key.as_str(), val, vm)?; - // } - // } - // - // for (key, val) in scope.locals { - // let key: PyRef = key.downcast().expect("Is a string"); - // - // if !logging_dict.contains_key(key.as_str(), vm) { - // logging_dict.set_item(key.as_str(), val, vm)?; - // } - // } - // } - Ok(scope.globals.to_pyobject(vm)) } -- cgit 1.4.1