Wt

Namaskara Roel,

I noticed this fact.

I read the code and in the constructor and applyValidationStyle()

WApplication *app = WApplication::instance();

Now WApplication::instance() is a static function.

I abhor static functions.

I then modified the WBoostrapTheme where the WApplication pointer is passed as C'tor parameter.

WBootstrapTheme(Wt::WApplication*      in_app_ptr,
                Wt::BootstrapVersion   in_bootstrapversion,
                bool                   in_responsive,
                bool                   in_formControlStyle
               );

I store the Application* as private member variable.

I tested the above C'tor. It worked beautifully.

Since I set the Bootstrap version, responsiveness and formcontrolstyle in c'tor itself, my code is that much compact and robust.

I dont have to set the WApplication* repeatedly.

Maybe this helps.

Further, I would replace Shared_ptr with unique_ptr& that way the invocation is that much quicker. Please give that a thought.

Regards
Rathnadhar KV

WApplication::instance() returns a pointer to the current WApplication in the current thread (we use a thread local variable for that), so it's not as nasty as you think it may be.