21128658087

Committed 19 Jan 2026 07:17AM UTC coverage: 60.506% (+0.001%) from 60.505%

Build # 21128658087

Build Type

push

github

Committed by

web-flow

Commit Message

Tidy copy initialization  (#859)

* trigger tidy

* copy&move BApplicPtr obj
always commit tidy changes

* move PolObject

* fix house add_component

* Automated clang-tidy change: performance-unnecessary-copy-initialization,performance-unnecessary-value-param

* Revert "Automated clang-tidy change: performance-unnecessary-copy-initialization,performance-unnecessary-value-param"

This reverts commit bb6aab13d.

* performance-unnecessary-value-param gives some weird suggestions

* Automated clang-tidy change: performance-unnecessary-copy-initialization

* minor performance improvements

* added used tidy check

* missing dependency

* renamed build so its no longer requires?

* better add docs for the sneaky fix

* silence pr check, not all headers can be compiled seperatly

---------

Co-authored-by: Clang Tidy <clang-tidy@users.noreply.github.com>

Run Details

33 of 41 new or added lines in 26 files covered. (80.49%)

2 existing lines in 1 file now uncovered.

44449 of 73462 relevant lines covered (60.51%)

529548.3 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

78.16

/pol-core/bscript/bstruct.cpp

/** @file
 *
 * @par History
 * - 2005/11/26 Shinigami: changed "strcmp" into "stricmp" to suppress Script Errors
 * - 2008/02/11 Turley:    BStruct::unpack() will accept zero length Structs
 * - 2009/09/05 Turley:    Added struct .? and .- as shortcut for .exists() and .erase()
 * - 2009/12/21 Turley:    ._method() call fix
 */


#include "bstruct.h"

#include <stddef.h>

#include "../clib/passert.h"
#include "../clib/stlutil.h"
#include "berror.h"
#include "bobject.h"
#include "contiter.h"
#include "executor.h"
#include "impstr.h"
#include "objmethods.h"


namespace Pol::Bscript
{
BStruct::BStruct() : BObjectImp( OTStruct ), contents_() {}

BStruct::BStruct( BObjectType type ) : BObjectImp( type ), contents_() {}

BStruct::BStruct( const BStruct& other, BObjectType type ) : BObjectImp( type ), contents_()
{
  for ( const auto& elem : other.contents_ )
  {
    const std::string& key = elem.first;
    const BObjectRef& bvalref = elem.second;

    contents_[key] = BObjectRef( new BObject( bvalref->impref().copy() ) );
  }
}

BStruct::BStruct( std::istream& is, unsigned size, BObjectType type )
    : BObjectImp( type ), contents_()
{
  for ( unsigned i = 0; i < size; ++i )
  {
    BObjectImp* keyimp = BObjectImp::unpack( is );
    BObjectImp* valimp = BObjectImp::unpack( is );
    if ( auto* key = impptrIf<String>( keyimp ); valimp && key )
    {
      contents_[key->value()].set( new BObject( valimp ) );

      BObject cleaner( key );
    }
    else
    {
      if ( keyimp )
        BObject objk( keyimp );
      if ( valimp )
        BObject objv( valimp );
    }
  }
}

BObjectImp* BStruct::copy() const
{
  passert( isa( OTStruct ) );
  return new BStruct( *this, OTStruct );
}
char BStruct::packtype() const
{
  return 't';
}

const char* BStruct::typetag() const
{
  return "struct";
}

const char* BStruct::typeOf() const
{
  return "Struct";
}
u8 BStruct::typeOfInt() const
{
  return OTStruct;
}


BObjectImp* BStruct::unpack( std::istream& is )
{
  unsigned size;
  char colon;
  if ( !( is >> size >> colon ) )
  {
    return new BError( "Unable to unpack struct elemcount" );
  }
  if ( (int)size < 0 )
  {
    return new BError(
        "Unable to unpack struct elemcount. Length given must be positive integer!" );
  }
  if ( colon != ':' )
  {
    return new BError( "Unable to unpack struct elemcount. Bad format. Colon not found!" );
  }
  return new BStruct( is, size, OTStruct );
}

void BStruct::FormatForStringRep( std::ostream& os, const std::string& key,
                                  const BObjectRef& bvalref ) const
{
  os << key << " = " << bvalref->impref().getFormattedStringRep();
}

class BStructIterator final : public ContIterator
{
public:
  BStructIterator( BStruct* pDict, BObject* pIterVal );
  BObject* step() override;

private:
  BObject m_StructObj;
  BStruct* m_pStruct;
  BObjectRef m_IterVal;
  std::string key;
  bool m_First;
};
BStructIterator::BStructIterator( BStruct* pStruct, BObject* pIterVal )
    : m_StructObj( pStruct ),
      m_pStruct( pStruct ),
      m_IterVal( pIterVal ),
      key( "" ),
      m_First( true )
{
}

BObject* BStructIterator::step()
{
  if ( m_First )
  {
    auto itr = m_pStruct->contents_.begin();
    if ( itr == m_pStruct->contents_.end() )
      return nullptr;

    m_First = false;
    key = ( *itr ).first;
    m_IterVal->setimp( new String( key ) );

    BObjectRef& oref = ( *itr ).second;
    return oref.get();
  }

  auto itr = m_pStruct->contents_.find( key );
  if ( itr == m_pStruct->contents_.end() )
    return nullptr;
  ++itr;
  if ( itr == m_pStruct->contents_.end() )
    return nullptr;

  key = ( *itr ).first;
  m_IterVal->setimp( new String( key ) );

  BObjectRef& oref = ( *itr ).second;
  return oref.get();
}

ContIterator* BStruct::createIterator( BObject* pIterVal )
{
  return new BStructIterator( this, pIterVal );
}

size_t BStruct::sizeEstimate() const
{
  return sizeof( BStruct ) +
         Clib::memsize( contents_, []( const auto& v ) { return v.sizeEstimate(); } );
}

size_t BStruct::mapcount() const
{
  return contents_.size();
}


BObjectRef BStruct::set_member( const char* membername, BObjectImp* value, bool copy )
{
  BObjectImp* target = copy ? value->copy() : value;
  auto itr = contents_.find( membername );
  if ( itr != contents_.end() )
  {
    BObjectRef& oref = ( *itr ).second;
    oref->setimp( target );
    return oref;
  }
  BObjectRef ref( new BObject( target ) );
  contents_.emplace( membername, ref );
  return ref;
}

// used programmatically
const BObjectImp* BStruct::FindMember( const char* name )
{
  auto itr = contents_.find( name );
  if ( itr != contents_.end() )
    return ( *itr ).second->impptr();
  return nullptr;
}

BObjectRef BStruct::get_member( const char* membername )
{
  auto itr = contents_.find( membername );
  if ( itr != contents_.end() )
    return ( *itr ).second;
  return BObjectRef( UninitObject::create() );
}

BObjectRef BStruct::OperSubscript( const BObject& obj )
{
  if ( obj->isa( OTString ) )
  {
    const String* keystr = obj.impptr<String>();

    auto itr = contents_.find( keystr->value() );
    if ( itr != contents_.end() )
    {
      BObjectRef& oref = ( *itr ).second;
      return oref;
    }
    return BObjectRef( UninitObject::create() );
  }
  return BObjectRef( new BError( "Struct members can only be accessed by name" ) );
}

BObjectImp* BStruct::array_assign( BObjectImp* idx, BObjectImp* target, bool copy )
{
  if ( auto* key = impptrIf<String>( idx ) )
  {
    BObjectImp* new_target = copy ? target->copy() : target;

    auto itr = contents_.find( key->value() );
    if ( itr != contents_.end() )
    {
      BObjectRef& oref = ( *itr ).second;
      oref->setimp( new_target );
      return new_target;
    }
    contents_[key->value()].set( new BObject( new_target ) );
    return new_target;
  }
  return new BError( "Struct members can only be accessed by name" );
}

void BStruct::addMember( const char* name, BObjectRef val )
{
  contents_[name] = std::move( val );
}

void BStruct::addMember( const char* name, BObjectImp* imp )
{
  contents_[name] = BObjectRef( imp );
}

BObjectImp* BStruct::call_method_id( const int id, Executor& ex, bool /*forcebuiltin*/ )
{
  BObject* keyobj;
  BObject* valobj;
  switch ( id )
  {
  case MTH_SIZE:
    if ( ex.numParams() == 0 )
      return new BLong( static_cast<int>( contents_.size() ) );
    return new BError( "struct.size() doesn't take parameters." );

  case MTH_ERASE:
    if ( ex.numParams() == 1 && ( keyobj = ex.getParamObj( 0 ) ) != nullptr )
    {
      if ( !keyobj->isa( OTString ) )
        return new BError( "Struct keys must be strings" );
      String* strkey = keyobj->impptr<String>();
      int nremove = static_cast<int>( contents_.erase( strkey->value() ) );
      return new BLong( nremove );
    }
    return new BError( "struct.erase(key) requires a parameter." );
    break;
  case MTH_INSERT:
    if ( ex.numParams() == 2 && ( keyobj = ex.getParamObj( 0 ) ) != nullptr &&
         ( valobj = ex.getParamObj( 1 ) ) != nullptr )
    {
      if ( !keyobj->isa( OTString ) )
        return new BError( "Struct keys must be strings" );
      String* strkey = keyobj->impptr<String>();
      contents_[strkey->value()] = BObjectRef( new BObject( valobj->impptr()->copy() ) );
      return new BLong( static_cast<int>( contents_.size() ) );
    }
    return new BError( "struct.insert(key,value) requires two parameters." );
    break;
  case MTH_EXISTS:
    if ( ex.numParams() == 1 && ( keyobj = ex.getParamObj( 0 ) ) != nullptr )
    {
      if ( !keyobj->isa( OTString ) )
        return new BError( "Struct keys must be strings" );
      String* strkey = keyobj->impptr<String>();
      int count = static_cast<int>( contents_.count( strkey->value() ) );
      return new BLong( count );
    }
    return new BError( "struct.exists(key) requires a parameter." );

  case MTH_KEYS:
    if ( ex.numParams() == 0 )
    {
      std::unique_ptr<ObjArray> arr( new ObjArray );
      for ( const auto& content : contents_ )
      {
        arr->addElement( new String( content.first ) );
      }
      return arr.release();
    }
    return new BError( "struct.keys() doesn't take parameters." );
    break;
  default:
    return nullptr;
  }
}

BObjectImp* BStruct::call_method( const char* methodname, Executor& ex )
{
  ObjMethod* objmethod = getKnownObjMethod( methodname );
  if ( objmethod != nullptr )
    return this->call_method_id( objmethod->id, ex );
  return nullptr;
}

void BStruct::packonto( std::ostream& os ) const
{
  os << packtype() << contents_.size() << ":";
  for ( const auto& content : contents_ )
  {
    const std::string& key = content.first;
    const BObjectRef& bvalref = content.second;

    String::packonto( os, key );
    bvalref->impref().packonto( os );
  }
}

std::string BStruct::getStringRep() const
{
  OSTRINGSTREAM os;
  os << typetag() << "{ ";
  bool any = false;

  for ( const auto& content : contents_ )
  {
    const std::string& key = content.first;
    const BObjectRef& bvalref = content.second;

    if ( any )
      os << ", ";
    else
      any = true;

    FormatForStringRep( os, key, bvalref );
  }

  os << " }";

  return OSTRINGSTREAM_STR( os );
}


BObjectRef BStruct::operDotPlus( const char* name )
{
  if ( contents_.count( name ) == 0 )
  {
    auto pnewobj = new BObject( new UninitObject );
    contents_[name] = BObjectRef( pnewobj );
    return BObjectRef( pnewobj );
  }
  return BObjectRef( new BError( "Member already exists" ) );
}

BObjectRef BStruct::operDotMinus( const char* name )
{
  contents_.erase( name );
  return BObjectRef( new BLong( 1 ) );
}

BObjectRef BStruct::operDotQMark( const char* name )
{
  int count = static_cast<int>( contents_.count( name ) );
  return BObjectRef( new BLong( count ) );
}

const BStruct::Contents& BStruct::contents() const
{
  return contents_;
}
}  // namespace Pol::Bscript

1	/** @file
2	*
3	* @par History
4	* - 2005/11/26 Shinigami: changed "strcmp" into "stricmp" to suppress Script Errors
5	* - 2008/02/11 Turley: BStruct::unpack() will accept zero length Structs
6	* - 2009/09/05 Turley: Added struct .? and .- as shortcut for .exists() and .erase()
7	* - 2009/12/21 Turley: ._method() call fix
8	*/
9
10
11	#include "bstruct.h"
12
13	#include <stddef.h>
14
15	#include "../clib/passert.h"
16	#include "../clib/stlutil.h"
17	#include "berror.h"
18	#include "bobject.h"
19	#include "contiter.h"
20	#include "executor.h"
21	#include "impstr.h"
22	#include "objmethods.h"
23
24
25	namespace Pol::Bscript
26	{
27	BStruct::BStruct() : BObjectImp( OTStruct ), contents_() {}	3,053✔
28
29	BStruct::BStruct( BObjectType type ) : BObjectImp( type ), contents_() {}	6,155✔
30
31	BStruct::BStruct( const BStruct& other, BObjectType type ) : BObjectImp( type ), contents_()	2,672✔
32	{
33	for ( const auto& elem : other.contents_ )	9,646✔
34	{
35	const std::string& key = elem.first;	6,974✔
36	const BObjectRef& bvalref = elem.second;	6,974✔
37
38	contents_[key] = BObjectRef( new BObject( bvalref->impref().copy() ) );	6,974✔
39	}
40	}	2,672✔
41
42	BStruct::BStruct( std::istream& is, unsigned size, BObjectType type )	3✔
43	: BObjectImp( type ), contents_()	3✔
44	{
45	for ( unsigned i = 0; i < size; ++i )	6✔
46	{
47	BObjectImp* keyimp = BObjectImp::unpack( is );	3✔
48	BObjectImp* valimp = BObjectImp::unpack( is );	3✔
49	if ( auto* key = impptrIf<String>( keyimp ); valimp && key )	3✔
50	{
51	contents_[key->value()].set( new BObject( valimp ) );	3✔
52
53	BObject cleaner( key );	3✔
54	}	3✔
55	else
56	{
57	if ( keyimp )	×
58	BObject objk( keyimp );	×
59	if ( valimp )	×
60	BObject objv( valimp );	×
61	}
62	}
63	}	3✔
64
65	BObjectImp* BStruct::copy() const	2,566✔
66	{
67	passert( isa( OTStruct ) );	2,566✔
68	return new BStruct( *this, OTStruct );	2,566✔
69	}
70	char BStruct::packtype() const	6✔
71	{
72	return 't';	6✔
73	}
74
75	const char* BStruct::typetag() const	1,611✔
76	{
77	return "struct";	1,611✔
78	}
79
80	const char* BStruct::typeOf() const	13✔
81	{
82	return "Struct";	13✔
83	}
84	u8 BStruct::typeOfInt() const	6✔
85	{
86	return OTStruct;	6✔
87	}
88
89
90	BObjectImp* BStruct::unpack( std::istream& is )	×
91	{
92	unsigned size;
93	char colon;
94	if ( !( is >> size >> colon ) )	×
95	{
96	return new BError( "Unable to unpack struct elemcount" );	×
97	}
98	if ( (int)size < 0 )	×
99	{
100	return new BError(
101	"Unable to unpack struct elemcount. Length given must be positive integer!" );	×
102	}
103	if ( colon != ':' )	×
104	{
105	return new BError( "Unable to unpack struct elemcount. Bad format. Colon not found!" );	×
106	}
107	return new BStruct( is, size, OTStruct );	×
108	}
109
110	void BStruct::FormatForStringRep( std::ostream& os, const std::string& key,	3,088✔
111	const BObjectRef& bvalref ) const
112	{
113	os << key << " = " << bvalref->impref().getFormattedStringRep();	3,088✔
114	}	3,088✔
115
116	class BStructIterator final : public ContIterator
117	{
118	public:
119	BStructIterator( BStruct* pDict, BObject* pIterVal );
120	BObject* step() override;
121
122	private:
123	BObject m_StructObj;
124	BStruct* m_pStruct;
125	BObjectRef m_IterVal;
126	std::string key;
127	bool m_First;
128	};
129	BStructIterator::BStructIterator( BStruct* pStruct, BObject* pIterVal )	61✔
130	: m_StructObj( pStruct ),	61✔
131	m_pStruct( pStruct ),	61✔
132	m_IterVal( pIterVal ),	61✔
133	key( "" ),	61✔
134	m_First( true )	61✔
135	{
136	}	61✔
137
138	BObject* BStructIterator::step()	221✔
139	{
140	if ( m_First )	221✔
141	{
142	auto itr = m_pStruct->contents_.begin();	61✔
143	if ( itr == m_pStruct->contents_.end() )	61✔
144	return nullptr;	×
145
146	m_First = false;	61✔
147	key = ( *itr ).first;	61✔
148	m_IterVal->setimp( new String( key ) );	61✔
149
150	BObjectRef& oref = ( *itr ).second;	61✔
151	return oref.get();	61✔
152	}
153
154	auto itr = m_pStruct->contents_.find( key );	160✔
155	if ( itr == m_pStruct->contents_.end() )	160✔
156	return nullptr;	×
157	++itr;	160✔
158	if ( itr == m_pStruct->contents_.end() )	160✔
159	return nullptr;	61✔
160
161	key = ( *itr ).first;	99✔
162	m_IterVal->setimp( new String( key ) );	99✔
163
164	BObjectRef& oref = ( *itr ).second;	99✔
165	return oref.get();	99✔
166	}
167
168	ContIterator* BStruct::createIterator( BObject* pIterVal )	61✔
169	{
170	return new BStructIterator( this, pIterVal );	61✔
171	}
172
173	size_t BStruct::sizeEstimate() const	343✔
174	{
175	return sizeof( BStruct ) +
176	Clib::memsize( contents_, []( const auto& v ) { return v.sizeEstimate(); } );	776✔
177	}
178
179	size_t BStruct::mapcount() const	3✔
180	{
181	return contents_.size();	3✔
182	}
183
184
185	BObjectRef BStruct::set_member( const char* membername, BObjectImp* value, bool copy )	608✔
186	{
187	BObjectImp* target = copy ? value->copy() : value;	608✔
188	auto itr = contents_.find( membername );	608✔
189	if ( itr != contents_.end() )	608✔
190	{
191	BObjectRef& oref = ( *itr ).second;	187✔
192	oref->setimp( target );	187✔
193	return oref;	187✔
194	}
195	BObjectRef ref( new BObject( target ) );	421✔
196	contents_.emplace( membername, ref );	421✔
197	return ref;	421✔
198	}	421✔
199
200	// used programmatically
201	const BObjectImp* BStruct::FindMember( const char* name )	50✔
202	{
203	auto itr = contents_.find( name );	50✔
204	if ( itr != contents_.end() )	50✔
205	return ( *itr ).second->impptr();	13✔
206	return nullptr;	37✔
207	}
208
209	BObjectRef BStruct::get_member( const char* membername )	11,767✔
210	{
211	auto itr = contents_.find( membername );	11,767✔
212	if ( itr != contents_.end() )	11,767✔
213	return ( *itr ).second;	11,711✔
214	return BObjectRef( UninitObject::create() );	56✔
215	}
216
217	BObjectRef BStruct::OperSubscript( const BObject& obj )	276✔
218	{
219	if ( obj->isa( OTString ) )	276✔
220	{
221	const String* keystr = obj.impptr<String>();	273✔
222
223	auto itr = contents_.find( keystr->value() );	273✔
224	if ( itr != contents_.end() )	273✔
225	{
226	BObjectRef& oref = ( *itr ).second;	270✔
227	return oref;	270✔
228	}
229	return BObjectRef( UninitObject::create() );	3✔
230	}
231	return BObjectRef( new BError( "Struct members can only be accessed by name" ) );	3✔
232	}
233
234	BObjectImp* BStruct::array_assign( BObjectImp* idx, BObjectImp* target, bool copy )	38✔
235	{
236	if ( auto* key = impptrIf<String>( idx ) )	38✔
237	{
238	BObjectImp* new_target = copy ? target->copy() : target;	35✔
239
240	auto itr = contents_.find( key->value() );	35✔
241	if ( itr != contents_.end() )	35✔
242	{
243	BObjectRef& oref = ( *itr ).second;	15✔
244	oref->setimp( new_target );	15✔
245	return new_target;	15✔
246	}
247	contents_[key->value()].set( new BObject( new_target ) );	20✔
248	return new_target;	20✔
249	}
250	return new BError( "Struct members can only be accessed by name" );	3✔
251	}
252
253	void BStruct::addMember( const char* name, BObjectRef val )	×
254	{
NEW 255	contents_[name] = std::move( val );	×
256	}	×
257
258	void BStruct::addMember( const char* name, BObjectImp* imp )	9,235✔
259	{
260	contents_[name] = BObjectRef( imp );	9,235✔
261	}	9,235✔
262
263	BObjectImp* BStruct::call_method_id( const int id, Executor& ex, bool /forcebuiltin/ )	670✔
264	{
265	BObject* keyobj;
266	BObject* valobj;
267	switch ( id )	670✔
268	{
269	case MTH_SIZE:	27✔
270	if ( ex.numParams() == 0 )	27✔
271	return new BLong( static_cast<int>( contents_.size() ) );	27✔
272	return new BError( "struct.size() doesn't take parameters." );	×
273
274	case MTH_ERASE:	×
275	if ( ex.numParams() == 1 && ( keyobj = ex.getParamObj( 0 ) ) != nullptr )	×
276	{
277	if ( !keyobj->isa( OTString ) )	×
278	return new BError( "Struct keys must be strings" );	×
279	String* strkey = keyobj->impptr<String>();	×
280	int nremove = static_cast<int>( contents_.erase( strkey->value() ) );	×
281	return new BLong( nremove );	×
282	}
283	return new BError( "struct.erase(key) requires a parameter." );	×
284	break;
285	case MTH_INSERT:	15✔
286	if ( ex.numParams() == 2 && ( keyobj = ex.getParamObj( 0 ) ) != nullptr &&	21✔
287	( valobj = ex.getParamObj( 1 ) ) != nullptr )	6✔
288	{
289	if ( !keyobj->isa( OTString ) )	6✔
290	return new BError( "Struct keys must be strings" );	×
291	String* strkey = keyobj->impptr<String>();	6✔
292	contents_[strkey->value()] = BObjectRef( new BObject( valobj->impptr()->copy() ) );	6✔
293	return new BLong( static_cast<int>( contents_.size() ) );	6✔
294	}
295	return new BError( "struct.insert(key,value) requires two parameters." );	9✔
296	break;
297	case MTH_EXISTS:	625✔
298	if ( ex.numParams() == 1 && ( keyobj = ex.getParamObj( 0 ) ) != nullptr )	625✔
299	{
300	if ( !keyobj->isa( OTString ) )	625✔
301	return new BError( "Struct keys must be strings" );	×
302	String* strkey = keyobj->impptr<String>();	625✔
303	int count = static_cast<int>( contents_.count( strkey->value() ) );	625✔
304	return new BLong( count );	625✔
305	}
306	return new BError( "struct.exists(key) requires a parameter." );	×
307
308	case MTH_KEYS:	×
309	if ( ex.numParams() == 0 )	×
310	{
311	std::unique_ptr<ObjArray> arr( new ObjArray );	×
312	for ( const auto& content : contents_ )	×
313	{
314	arr->addElement( new String( content.first ) );	×
315	}
316	return arr.release();	×
317	}	×
318	return new BError( "struct.keys() doesn't take parameters." );	×
319	break;
320	default:	3✔
321	return nullptr;	3✔
322	}
323	}
324
325	BObjectImp* BStruct::call_method( const char* methodname, Executor& ex )	×
326	{
327	ObjMethod* objmethod = getKnownObjMethod( methodname );	×
328	if ( objmethod != nullptr )	×
329	return this->call_method_id( objmethod->id, ex );	×
330	return nullptr;	×
331	}
332
333	void BStruct::packonto( std::ostream& os ) const	9✔
334	{
335	os << packtype() << contents_.size() << ":";	9✔
336	for ( const auto& content : contents_ )	24✔
337	{
338	const std::string& key = content.first;	15✔
339	const BObjectRef& bvalref = content.second;	15✔
340
341	String::packonto( os, key );	15✔
342	bvalref->impref().packonto( os );	15✔
343	}
344	}	9✔
345
346	std::string BStruct::getStringRep() const	2,246✔
347	{
348	OSTRINGSTREAM os;	2,246✔
349	os << typetag() << "{ ";	2,246✔
350	bool any = false;	2,246✔
351
352	for ( const auto& content : contents_ )	5,334✔
353	{
354	const std::string& key = content.first;	3,088✔
355	const BObjectRef& bvalref = content.second;	3,088✔
356
357	if ( any )	3,088✔
358	os << ", ";	1,292✔
359	else
360	any = true;	1,796✔
361
362	FormatForStringRep( os, key, bvalref );	3,088✔
363	}
364
365	os << " }";	2,246✔
366
367	return OSTRINGSTREAM_STR( os );	4,492✔
368	}	2,246✔
369
370
371	BObjectRef BStruct::operDotPlus( const char* name )	1,939✔
372	{
373	if ( contents_.count( name ) == 0 )	1,939✔
374	{
375	auto pnewobj = new BObject( new UninitObject );	1,933✔
376	contents_[name] = BObjectRef( pnewobj );	1,933✔
377	return BObjectRef( pnewobj );	1,933✔
378	}
379	return BObjectRef( new BError( "Member already exists" ) );	6✔
380	}
381
382	BObjectRef BStruct::operDotMinus( const char* name )	×
383	{
384	contents_.erase( name );	×
385	return BObjectRef( new BLong( 1 ) );	×
386	}
387
388	BObjectRef BStruct::operDotQMark( const char* name )	1,603✔
389	{
390	int count = static_cast<int>( contents_.count( name ) );	1,603✔
391	return BObjectRef( new BLong( count ) );	3,206✔
392	}
393
394	const BStruct::Contents& BStruct::contents() const	1,744✔
395	{
396	return contents_;	1,744✔
397	}
398	} // namespace Pol::Bscript

polserver / polserver / 21128658087

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous