The analogy I would make is to needing a helmet to ride a motorcycle. You don' t need it in the strictest sense, but you need it for a reasonable level of safety. You need RAII for a reasonable level of safety against resource leaks.
On Thu, Aug 31, 2023 at 1:50 PM Masakazu Kitajo <m4s...@gmail.com> wrote: > 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 . > > > > > > > > > >
