[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v4 12/14] simpletrace: added simplified Analyzer2 class
From: |
Daniel P . Berrangé |
Subject: |
Re: [PATCH v4 12/14] simpletrace: added simplified Analyzer2 class |
Date: |
Tue, 5 Sep 2023 13:42:46 +0100 |
User-agent: |
Mutt/2.2.9 (2022-11-12) |
On Wed, Aug 23, 2023 at 10:54:27AM +0200, Mads Ynddal wrote:
> From: Mads Ynddal <m.ynddal@samsung.com>
>
> By moving the dynamic argument construction to keyword-arguments,
> we can remove all of the specialized handling, and streamline it.
> If a tracing method wants to access these, they can define the
> kwargs, or ignore it be placing `**kwargs` at the end of the
> function's arguments list.
>
> Added deprecation warning to Analyzer class to make users aware
> of the Analyzer2 class. No removal date is planned.
AFAIK, we don't consider simpletrace.py python code to be a
supported public API, just a command line tool.
IOW, we can change the python code at will, as long as the
command line doesn't alter its behaviour. Thus I don't see
a need to add new classes, just change the existing ones.
With regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|