I see your point, but I think we need to draw a line. High-priority capability is blurry and questionable. We have not *needed* it for years.
> We could support RAII better, but I don't think it's feasible in 10.0. This implies Cleanup.h will be unnecessary after we make it better. Not discarding Cleanup.h and promoting it to TS API are different things. TS API is not something we can casually change or remove. I'd rather not add a short lifetime temporary solution to TS API. On Wed, Aug 30, 2023 at 12:19 PM Walt Karas <wka...@yahooinc.com.invalid> wrote: > 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 . > > > > > >
