breadcumbs queue is now a simple DisabledQueue when maxBreadcrumb is 0

if breadcumbs is a DisabledQueue, no more processing is done (beforeBreadcrumb or scopeObservers)

Author: stefanosiano

sentry/src/main/java/io/sentry/Scope.java

@@ -466,7 +466,7 @@ public Queue<Breadcrumb> getBreadcrumbs() {
    */
   @Override
   public void addBreadcrumb(@NotNull Breadcrumb breadcrumb, @Nullable Hint hint) {
-    if (breadcrumb == null) {
+    if (breadcrumb == null || breadcrumbs instanceof DisabledQueue) {
       return;
     }
     if (hint == null) {
@@ -862,7 +862,7 @@ public void clearAttachments() {
   static @NotNull Queue<Breadcrumb> createBreadcrumbsList(final int maxBreadcrumb) {
     return maxBreadcrumb > 0
         ? SynchronizedQueue.synchronizedQueue(new CircularFifoQueue<>(maxBreadcrumb))
-        : SynchronizedQueue.synchronizedQueue(new DisabledQueue<>());
+        : new DisabledQueue<>();
   }
 
   /**