yamt commented on pull request #695: URL: https://github.com/apache/incubator-nuttx-apps/pull/695#issuecomment-840969077
> > > it's something like having a unit test like the following and claiming that free() has a problem. > > > it doesn't make much sense. > > > ``` > > > p = malloc(1); > > > free(p); > > > free(p); > > > ``` > > > > > > It's completely different. > > If your comparison is based on the fact that I've called the `webclient_abort` twice, feel free to create another test that calls `webclient_perform` after a single `webclient_abort`. > > it's same. > > webclient_perform after webclient_abort is also same. > they are all wrong use of apis. > > you can invent more and more "problematic" sequences. eg. realloc after free. > but it doesn't mean these apis has problems. i added a dedicated comment block to explain webclient_context lifetime to explain the expected sequence of API calls. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
