The utilities in Cleanup.h add RAII capabilities to the TS API. I hope we don't need to have a debate about the value of RAII.
Certainly, any API can become bloated with too many features. But RAII seems like a high-priority capability to support. We could support RAII better, but I don't think it's feasible in 10.0. Remember, it's not really being added. We are removing the distinction between the C and C++ APIs. We' re discarding the parts of C++ API that are incompatible with the C API. I am proposing we not discard Cleanup.h, which is compatible and enhances (previously) C API. On Wed, Aug 30, 2023 at 1:15 PM Masakazu Kitajo <mas...@apache.org> wrote: > I wonder what should be part of TS API. There may be some exceptions, but > TS API basically provides things that can only be done on ATS core. > > Question for all, would we want to have utilities as TS API? > > -- Masakazu > > On Tue, Aug 22, 2023 at 11:48 AM Walt Karas <wka...@yahooinc.com.invalid> > wrote: > > > See the PR > https://urldefense.com/v3/__https://github.com/apache/trafficserver/pull/10231__;!!Op6eflyXZCqGR5I!HHlP-H_IgX9Ah46LlOiHqX--VcnzF3Dp5uG2JI_JXBmG8c_jMQNpsYQPvyDaaAsympzb7neltiG5Z7zseA$ > . Cleanup.h > > is currently available to plugins in include/tscpp/api . The proposal is > > to move it to include/ts . The declarations currently in the > > atscppapi namespace are moved to the tsapi::c_support namespace. The > > c_support sub-namespace is for declarations specifically designed to work > > with declarations in tsapi::c . > > >
