Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Activity
  • 15:30
    MHaselmaier synchronize #424
  • 15:20
    MHaselmaier synchronize #424
  • 15:20
    MHaselmaier synchronize #424
  • 15:16
    MHaselmaier synchronize #424
  • 14:59
    MHaselmaier synchronize #424
  • 14:36
    MHaselmaier edited #424
  • 14:36
    MHaselmaier synchronize #424
  • 13:14
    MHaselmaier edited #424
  • 13:13
    MHaselmaier edited #424
  • 10:15
    MHaselmaier synchronize #424
  • 10:07
    MHaselmaier synchronize #424
  • 10:03
    MHaselmaier synchronize #424
  • 09:59
    MHaselmaier synchronize #424
  • 09:55
    MHaselmaier synchronize #424
  • 00:38
    lganzzzo commented #424
  • May 12 20:32
    MHaselmaier synchronize #424
  • May 11 16:57
    lganzzzo commented #428
  • May 11 16:56

    lganzzzo on master

    Always use localtime_s on windo… Merge pull request #428 from au… (compare)

  • May 11 16:56
    lganzzzo closed #428
  • May 11 16:32
    Samega7Cattac commented #428
Bob Tolbert
@rwtolbert
here is another one
oat++
@oatpp_io_twitter
But I guess it might be better to simplify this section and put everything under the mutex. - I just have to make sure that performance will not degrade
Bob Tolbert
@rwtolbert
sure, i get that
Bob Tolbert
@rwtolbert
and this may be too much. I just discovered the Thread Sanitizer for some of my own code, so I thought I’d give it a go on the oat++ tests
oat++
@oatpp_io_twitter
Good, you found these warnings. I'll add an issue on github with your stack-traces
oat++
@oatpp_io_twitter
Hey @rwtolbert , I have just merged the branch with fixes for all Thread Sanitizer warnings
please find the PR - oatpp/oatpp#147
Also, it was almost zero performance impact on the Async processor.
Bob Tolbert
@rwtolbert
that is good news. I’ll take a look
Bob Tolbert
@rwtolbert
@oatpp_io_twitter Looks good. I put in a small PR for warnings from Clang on Mac
I'm now looking at warnings on Windows. They are almost all conversion warnings, for example:
oatpp::String BufferOutputStream::toString() {
  return oatpp::String((const char*)m_data, m_position, true);
}
m_position is v_io_size (ie v_int64) but the String ctor takes a v_int32
not sure a simple cast to silence the warning is a good idea, for example
oat++
@oatpp_io_twitter
@rwtolbert , yes, I was actually thinking of moving String size to v_int64
Bob Tolbert
@rwtolbert
makes sense
Bob Tolbert
@rwtolbert
a number of the other warnings can be fixed but require careful thought as well, not just casts. I will make a PR and you can have a look
oat++
@oatpp_io_twitter
Sounds great!
Bob Tolbert
@rwtolbert
i might do it in a few pieces, since there are a couple of classes of these. Do you want me to also make the String ctor to v_int64 change in this PR?
oat++
@oatpp_io_twitter

Do you want me to also make the String ctor to v_int64 change in this PR?

Yes, it would be cool!
Also, there are some pleases where explicit casts were applied to silent warnings... those should be deleted as well.
Unfortunately, I don't remember where exactly :)

It would be great to move all classed that use buffers to v_int64. Like MemoryLabel for example.

So I guess we can make it step-by-step. Because once you move String yo int64 to silence one bunch of warnings - others will pop up :))

... just like with any refactoring :))

oat++
@oatpp_io_twitter

@/all ,
Update :bell:

Oat++ version was increased to 0.19.10, and the new release tag is added.
Please find the release with changes made in October here 0.19.10

Release summary

  • ApiController. Add router prefix.
  • Introduce LazyStringMap to store headers and query parameters.
  • Introduce ConnectionPool. You may find usage example here. Used to increase performance and, in order to prevent ephemeral ports exhaustion on the client during high load.
  • Refactor. Rename ADDCORS --> ADD_CORS macro.
  • Optimization. Use BufferOutputStream in RequestHeadersReader.
  • Fix thread sanitizer warnings. - Special thanks to @rwtolbert !
  • Fix some compiler warnings.

Cheers :tada:
Leonid

Hey @ke9 , please find the oatpp/oatpp#149 I've added to track the progress on vcpkg support.
oat++
@oatpp_io_twitter
@/all - one more update. FYI - I started to work on oatpp/oatpp#150 - support for retires in RequestExecutor for ApiClient.
pooya
@pouyapanahandeh
Hi, I have some basic questions, the most important one is: how we can create the project, is it like Django, has any command line to create a project or not? I check the website but couldn't find anything, it would be great if someone explains it to me.
oat++
@oatpp_io_twitter
Hello @pouyapanahandeh , please checkout the starter project https://github.com/oatpp/oatpp-starter
This is a basic project with hello world endpoint
All you need is install oatpp. Checkout the starter and modify it to your needs
Please let me know if you have more questions
pooya
@pouyapanahandeh
@oatpp_io_twitter thanks for your help,
For example in Django we have the following command to create a project:
django-admin startproject mysite
How we can create the project with oatpp, I checked the link but I didn't get it well. you mean we have to clone the repo and use it as a starter project or it's an example of how the project looks like in oatpp?
oat++
@oatpp_io_twitter
Yes, you are right - just clone the stater and modify it to your needs.
Currently there is no oatpp-admin like in django. But it is a good idea to have one in future
@pouyapanahandeh
Starter is just a boilerplate precreated for you to start easier
pooya
@pouyapanahandeh
now I got it, one more question, do you have any doc how to set up the web app that written in oatpp on AWS or Azure?
oat++
@oatpp_io_twitter
Yes, there is no specific env setup for clouds. You can find some info here https://oatpp.io/docs/installation/unix-linux/
Basically you have to install c++ compiler and oatpp
Build your app and its done
However there might be some additional things depending on the OS you choose
pooya
@pouyapanahandeh
I got it, Thanks a lot for your help and time
oat++
@oatpp_io_twitter
Just text me if you face any problems. It would be helpfull for the project to know all the potential issues
pooya
@pouyapanahandeh
Sure, I will
Panda805
@Panda805
Hi ,there.
Can you tell me your por in detail?
oat++
@oatpp_io_twitter
Hello @Panda805 ,
Please specify what exactly you want to know?
I'll be happy to share the info
Marco Leclerc
@ke9
hi, anyone familiar with oatpp-consul?
oat++
@oatpp_io_twitter
Hello @ke9 what do you need a help with ?