Fix Log to emit with exception even if block outputs nil (#15253)

Previous change #12000 skips logging when the given block outputs `nil`, however it inadvertently also suppresses logging exceptions.
Logging is now skipped only if both the given exception and the block result are `nil`.

spec/std/log/log_spec.cr

@@ -264,13 +264,28 @@ describe Log do
       entry.exception.should be_nil
     end
 
-    it "does not emit anything when a nil is emitted" do
+    it "does not emit when block returns nil" do
       backend = Log::MemoryBackend.new
       log = Log.new("a", backend, :notice)
 
       log.notice { nil }
 
       backend.entries.should be_empty
     end
+
+    it "does emit when block returns nil but exception is provided" do
+      backend = Log::MemoryBackend.new
+      log = Log.new("a", backend, :notice)
+      ex = Exception.new "the attached exception"
+
+      log.notice(exception: ex) { nil }
+
+      entry = backend.entries.first
+      entry.source.should eq("a")
+      entry.severity.should eq(s(:notice))
+      entry.message.should eq("")
+      entry.data.should eq(Log::Metadata.empty)
+      entry.exception.should eq(ex)
+    end
   end
 end

src/log/log.cr

@@ -65,13 +65,16 @@ class Log
       dsl = Emitter.new(@source, severity, exception)
       result = yield dsl
 
-      case result
-      when Entry
-        backend.dispatch result
-      when Nil
-        # emit nothing
-      else
-        backend.dispatch dsl.emit(result.to_s)
+      unless result.nil? && exception.nil?
+        entry =
+           case result
+           when Entry
+             result
+           else
+             dsl.emit(result.to_s)
+           end
+
+        backend.dispatch entry
       end
     end
   {% end %}