Skip to content

Commit 03f588a

Browse files
committed
attempt to fix compilation on Linux
1 parent 06dff6d commit 03f588a

File tree

5 files changed

+12
-17
lines changed

5 files changed

+12
-17
lines changed

contract-tests/client-contract-tests/src/entity_manager.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,8 +41,7 @@ std::optional<std::string> EntityManager::create(ConfigParams const& in) {
4141
if (in.proxy) {
4242
if (in.proxy->httpProxy) {
4343
config_builder.HttpProperties().Proxy(
44-
HttpPropertiesBuilder::ProxyBuilder().HttpProxy(
45-
*in.proxy->httpProxy));
44+
ProxyBuilder().HttpProxy(*in.proxy->httpProxy));
4645
}
4746
}
4847

libs/client-sdk/src/bindings/c/builder.cpp

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -41,14 +41,12 @@ using namespace launchdarkly::client_side;
4141
#define FROM_CUSTOM_PERSISTENCE_BUILDER(ptr) \
4242
(reinterpret_cast<LDPersistenceCustomBuilder>(ptr))
4343

44-
#define TO_TLS_BUILDER(ptr) \
45-
(reinterpret_cast<HttpPropertiesBuilder::TlsBuilder*>(ptr))
44+
#define TO_TLS_BUILDER(ptr) (reinterpret_cast<TlsBuilder*>(ptr))
4645

4746
#define FROM_TLS_BUILDER(ptr) \
4847
(reinterpret_cast<LDClientHttpPropertiesTlsBuilder>(ptr))
4948

50-
#define TO_PROXY_BUILDER(ptr) \
51-
(reinterpret_cast<HttpPropertiesBuilder::ProxyBuilder*>(ptr))
49+
#define TO_PROXY_BUILDER(ptr) (reinterpret_cast<ProxyBuilder*>(ptr))
5250

5351
#define FROM_PROXY_BUILDER(ptr) \
5452
(reinterpret_cast<LDClientHttpPropertiesProxyBuilder>(ptr))
@@ -320,7 +318,7 @@ LDClientConfigBuilder_HttpProperties_Header(LDClientConfigBuilder b,
320318

321319
LD_EXPORT(LDClientHttpPropertiesProxyBuilder)
322320
LDClientHttpPropertiesProxyBuilder_New(void) {
323-
return FROM_PROXY_BUILDER(new HttpPropertiesBuilder::ProxyBuilder());
321+
return FROM_PROXY_BUILDER(new ProxyBuilder());
324322
}
325323

326324
LD_EXPORT(void)
@@ -383,7 +381,7 @@ LDClientHttpPropertiesTlsBuilder_CustomCAFile(
383381

384382
LD_EXPORT(LDClientHttpPropertiesTlsBuilder)
385383
LDClientHttpPropertiesTlsBuilder_New(void) {
386-
return FROM_TLS_BUILDER(new HttpPropertiesBuilder::TlsBuilder());
384+
return FROM_TLS_BUILDER(new TlsBuilder());
387385
}
388386

389387
LD_EXPORT(void)

libs/common/include/launchdarkly/config/client.hpp

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@ using DataSourceBuilder = config::shared::builders::DataSourceBuilder<SDK>;
2323
using LoggingBuilder = config::shared::builders::LoggingBuilder;
2424
using PersistenceBuilder = config::shared::builders::PersistenceBuilder<SDK>;
2525
using TlsBuilder = config::shared::builders::TlsBuilder<SDK>;
26+
using ProxyBuilder = config::shared::builders::ProxyBuilder<SDK>;
2627

2728
using Config = config::Config<SDK>;
2829

libs/common/include/launchdarkly/config/shared/builders/http_properties_builder.hpp

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -116,9 +116,6 @@ class ProxyBuilder {
116116
template <typename SDK>
117117
class HttpPropertiesBuilder {
118118
public:
119-
using TlsBuilder = TlsBuilder<SDK>;
120-
using ProxyBuilder = ProxyBuilder<SDK>;
121-
122119
/**
123120
* Construct a new HttpPropertiesBuilder. The builder will use the default
124121
* properties based on the SDK type. Setting a property will override
@@ -222,14 +219,14 @@ class HttpPropertiesBuilder {
222219
* @param builder The TLS property builder.
223220
* @return A reference to this builder.
224221
*/
225-
HttpPropertiesBuilder& Tls(TlsBuilder builder);
222+
HttpPropertiesBuilder& Tls(TlsBuilder<SDK> builder);
226223

227224
/**
228225
*
229226
* @param builder Sets the builder for proxy properties.
230227
* @return A reference to this builder.
231228
*/
232-
HttpPropertiesBuilder& Proxy(ProxyBuilder builder);
229+
HttpPropertiesBuilder& Proxy(ProxyBuilder<SDK> builder);
233230

234231
[[nodiscard]] built::HttpProperties Build() const;
235232

@@ -241,8 +238,8 @@ class HttpPropertiesBuilder {
241238
std::string wrapper_name_;
242239
std::string wrapper_version_;
243240
std::map<std::string, std::string> base_headers_;
244-
TlsBuilder tls_;
245-
ProxyBuilder proxy_;
241+
TlsBuilder<SDK> tls_;
242+
ProxyBuilder<SDK> proxy_;
246243
};
247244

248245
} // namespace launchdarkly::config::shared::builders

libs/common/src/config/http_properties_builder.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -143,14 +143,14 @@ HttpPropertiesBuilder<SDK>& HttpPropertiesBuilder<SDK>::Header(
143143

144144
template <typename SDK>
145145
HttpPropertiesBuilder<SDK>& HttpPropertiesBuilder<SDK>::Tls(
146-
TlsBuilder builder) {
146+
TlsBuilder<SDK> builder) {
147147
tls_ = std::move(builder);
148148
return *this;
149149
}
150150

151151
template <typename SDK>
152152
HttpPropertiesBuilder<SDK>& HttpPropertiesBuilder<SDK>::Proxy(
153-
ProxyBuilder builder) {
153+
ProxyBuilder<SDK> builder) {
154154
proxy_ = std::move(builder);
155155
return *this;
156156
}

0 commit comments

Comments
 (0)