Fix: MockSession example and default allocator [172764384]

[678098]

14NOV2023_17:13:53.982763 48988 8674795648 ERROR ~/blazingmq/src/groups/bmq/bmqa/bmqa_mocksession.cpp 113 UNINITIALIZED_LOGGER_MANAGER [ Description: 'Bad value for argument 'flags' in call to 'bmqa::OpenQueueStatus openQueueSync(QueueId *queueId,const bmqt::Uri& uri,bsls::Types::Uint64 flags,const bmqt::QueueOptions& options,const bsls::TimeInterval& timeout)': expected '2', got '10' (~/blazingmq/src/tutorials/subscriptions/consumer.cpp:857)', File: '~/blazingmq/src/tutorials/subscriptions/consumer.cpp:857' ] FATAL ~/blazingmq/src/groups/bmq/bmqa/bmqa_mocksession.cpp:123 Assertion failed: false zsh: abort ./consumer.tsk

[678098]

The problem was caused by this line. We have set expected flags == 2 (READER) here:

blazingmq/src/groups/bmq/bmqa/bmqa_mocksession.h

Lines 431 to 434 in 5ca3a4a

	// // Initialize the queue flags for a consumer
	// bsls::Types::Uint64 flags = 0;
	// bmqt::QueueFlagsUtil::setReader(&flags);
	//

And now we try to open queue with flags READER | ACK

[678098]

Fixed some places where new was called with allocator pointer not wrapped with bslma::Default::allocator. These places were causing crashes when allocator = 0 passed.

[pniedzielski]

Two independent changes, right? bslma::Default::allocator and avoiding the hardcoded value for flags.

LGTM

[678098]

@pniedzielski re-run clang-format

[pniedzielski]

@678098 on src/groups/bmq/bmqa/bmqa_mocksession.cpp as well :)

[678098]

@678098 on src/groups/bmq/bmqa/bmqa_mocksession.cpp as well :)

I formatted it with clang-format 16.0.6 and it makes a different formatting from CI

[678098]

llvm/llvm-project#54703
Looks like a fixed bug

[pniedzielski]

There's a failure in the CI that doesn't appear to have to do with this PR (it's failing on main as well, and doesn't appear on local builds). This PR is good to go, but we'll investigate that issue separately.