1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152 | /* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */
/*
* This file is part of the LibreOffice project.
*
* This Source Code Form is subject to the terms of the Mozilla Public
* License, v. 2.0. If a copy of the MPL was not distributed with this
* file, You can obtain one at http://mozilla.org/MPL/2.0/.
*
* This file incorporates work covered by the following license notice:
*
* Licensed to the Apache Software Foundation (ASF) under one or more
* contributor license agreements. See the NOTICE file distributed
* with this work for additional information regarding copyright
* ownership. The ASF licenses this file to you under the Apache
* License, Version 2.0 (the "License"); you may not use this file
* except in compliance with the License. You may obtain a copy of
* the License at http://www.apache.org/licenses/LICENSE-2.0 .
*/
#include <assert.h>
#include <vector>
#include "attributes.hxx"
namespace {
struct TagAttribute
{
TagAttribute( const OUString &rName, const OUString &rType , const OUString &rValue )
: sName(rName), sType(rType), sValue(rValue)
{
}
OUString sName;
OUString sType;
OUString sValue;
};
}
struct AttributeListImpl_impl
{
AttributeListImpl_impl()
{
// performance improvement during adding
vecAttribute.reserve(20);
}
std::vector<struct TagAttribute> vecAttribute;
};
sal_Int16 SAL_CALL AttributeListImpl::getLength()
{
return static_cast<sal_Int16>(m_pImpl->vecAttribute.size());
}
AttributeListImpl::AttributeListImpl( const AttributeListImpl &r )
: cppu::WeakImplHelper<css::xml::sax::XAttributeList>( r ),
m_pImpl( new AttributeListImpl_impl )
{
*m_pImpl = *(r.m_pImpl);
}
OUString AttributeListImpl::getNameByIndex(sal_Int16 i)
{
sal_uInt32 i2 = sal::static_int_cast<sal_Int16>(i);
if( i >= 0 && i2 < m_pImpl->vecAttribute.size() )
{
return m_pImpl->vecAttribute[i].sName;
}
return OUString();
}
OUString AttributeListImpl::getTypeByIndex(sal_Int16 i)
{
sal_uInt32 i2 = sal::static_int_cast<sal_Int16>(i);
if( i >= 0 && i2 < m_pImpl->vecAttribute.size() )
{
return m_pImpl->vecAttribute[i].sType;
}
return OUString();
}
OUString AttributeListImpl::getValueByIndex(sal_Int16 i)
{
sal_uInt32 i2 = sal::static_int_cast<sal_Int16>(i);
if( i >= 0 && i2 < m_pImpl->vecAttribute.size() )
{
return m_pImpl->vecAttribute[i].sValue;
}
return OUString();
}
OUString AttributeListImpl::getTypeByName( const OUString& sName )
{
for (auto const& elem : m_pImpl->vecAttribute)
{
if( elem.sName == sName )
{<--- Consider using std::find_if algorithm instead of a raw loop.
return elem.sType;
}
}
return OUString();
}
OUString AttributeListImpl::getValueByName(const OUString& sName)
{
for (auto const& elem : m_pImpl->vecAttribute)
{
if( elem.sName == sName )
{<--- Consider using std::find_if algorithm instead of a raw loop.
return elem.sValue;
}
}
return OUString();
}
AttributeListImpl::AttributeListImpl()
: m_pImpl( new AttributeListImpl_impl )
{
}
AttributeListImpl::~AttributeListImpl()
{
}
void AttributeListImpl::addAttribute( const OUString &sName ,
const OUString &sType ,
const OUString &sValue )
{
m_pImpl->vecAttribute.emplace_back( sName , sType , sValue );
}
void AttributeListImpl::clear()
{
std::vector<struct TagAttribute> dummy;
m_pImpl->vecAttribute.swap( dummy );
assert( ! getLength() );<--- Assert statement calls a function which may have desired side effects: 'getLength'. [+]Non-pure function: 'getLength' is called inside assert statement. Assert statements are removed from release builds so the code inside assert statement is not executed. If the code is needed also in release builds, this is a bug.
}
/* vim:set shiftwidth=4 softtabstop=4 expandtab: */
|