Skip to content

Commit f089616

Browse files
authored
perf: Use MOZ_ASSERT instead of MOZ_RELEASE_ASSERT as these methods are already guarded correctly where they are being called (#876)
1 parent 1820b0b commit f089616

File tree

1 file changed

+6
-5
lines changed

1 file changed

+6
-5
lines changed

runtime/fastly/builtins/cache-override.cpp

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -28,14 +28,15 @@ void CacheOverride::set_mode(JSObject *self, CacheOverride::CacheOverrideMode mo
2828

2929
JS::Value CacheOverride::ttl(JSObject *self) {
3030
MOZ_ASSERT(is_instance(self));
31-
if (CacheOverride::mode(self) != CacheOverride::CacheOverrideMode::Override)
31+
if (CacheOverride::mode(self) != CacheOverride::CacheOverrideMode::Override) {
3232
return JS::UndefinedValue();
33+
}
3334
return JS::GetReservedSlot(self, Slots::TTL);
3435
}
3536

3637
void CacheOverride::set_ttl(JSObject *self, uint32_t ttl) {
3738
MOZ_ASSERT(is_instance(self));
38-
MOZ_RELEASE_ASSERT(mode(self) == CacheOverride::CacheOverrideMode::Override);
39+
MOZ_ASSERT(mode(self) == CacheOverride::CacheOverrideMode::Override);
3940
JS::SetReservedSlot(self, CacheOverride::Slots::TTL, JS::Int32Value((int32_t)ttl));
4041
}
4142

@@ -48,7 +49,7 @@ JS::Value CacheOverride::swr(JSObject *self) {
4849

4950
void CacheOverride::set_swr(JSObject *self, uint32_t swr) {
5051
MOZ_ASSERT(is_instance(self));
51-
MOZ_RELEASE_ASSERT(CacheOverride::mode(self) == CacheOverride::CacheOverrideMode::Override);
52+
MOZ_ASSERT(CacheOverride::mode(self) == CacheOverride::CacheOverrideMode::Override);
5253
JS::SetReservedSlot(self, CacheOverride::Slots::SWR, JS::Int32Value((int32_t)swr));
5354
}
5455

@@ -61,7 +62,7 @@ JS::Value CacheOverride::surrogate_key(JSObject *self) {
6162

6263
void CacheOverride::set_surrogate_key(JSObject *self, JSString *key) {
6364
MOZ_ASSERT(is_instance(self));
64-
MOZ_RELEASE_ASSERT(CacheOverride::mode(self) == CacheOverride::CacheOverrideMode::Override);
65+
MOZ_ASSERT(CacheOverride::mode(self) == CacheOverride::CacheOverrideMode::Override);
6566
JS::SetReservedSlot(self, CacheOverride::Slots::SurrogateKey, JS::StringValue(key));
6667
}
6768

@@ -74,7 +75,7 @@ JS::Value CacheOverride::pci(JSObject *self) {
7475

7576
void CacheOverride::set_pci(JSObject *self, bool pci) {
7677
MOZ_ASSERT(is_instance(self));
77-
MOZ_RELEASE_ASSERT(CacheOverride::mode(self) == CacheOverride::CacheOverrideMode::Override);
78+
MOZ_ASSERT(CacheOverride::mode(self) == CacheOverride::CacheOverrideMode::Override);
7879
JS::SetReservedSlot(self, CacheOverride::Slots::PCI, JS::BooleanValue(pci));
7980
}
8081

0 commit comments

Comments
 (0)