fix compatibility with elixir 1.15 (#570)

Author: mat-hek

lib/membrane/core/child.ex

@@ -23,7 +23,7 @@ defmodule Membrane.Core.Child do
         doc -> doc
       end
 
-    if moduledoc != false do
+    if is_binary(moduledoc) and String.valid?(moduledoc) do
       moduledoc =
         if String.trim(moduledoc) == "" do
           "Membrane #{child_type}.\n\n"

lib/membrane/core/child/pads_specs.ex

@@ -54,7 +54,7 @@ defmodule Membrane.Core.Child.PadsSpecs do
 
     for format <- accepted_formats do
       with :any <- format do
-        Membrane.Logger.warn("""
+        Membrane.Logger.warning("""
         Remeber, that `accepted_format: :any` in pad definition will be satisified by stream format in form of %:any{}, \
         not >>any<< stream format (to achieve such an effect, put `accepted_format: _any` in your code)
         """)

lib/membrane/core/observability.ex

@@ -39,7 +39,9 @@ defmodule Membrane.Core.Observability do
         else: {pid_string, "", " (#{component_type})"}
 
     name_suffix = if component_type == :element, do: "", else: "/"
-    name_str = if(String.valid?(name), do: name, else: inspect(name)) <> name_suffix
+
+    name_str =
+      if(is_binary(name) and String.valid?(name), do: name, else: inspect(name)) <> name_suffix
 
     register_name_for_observer(
       :"##{unique_prefix}#{name_str}#{component_type_suffix}#{utility_name}"

lib/membrane/core/parent/child_life_controller.ex

@@ -479,7 +479,7 @@ defmodule Membrane.Core.Parent.ChildLifeController do
     {already_removing, data} = Enum.split_with(data, & &1.terminating?)
 
     if already_removing != [] do
-      Membrane.Logger.warn("""
+      Membrane.Logger.warning("""
       Trying to remove children that are already being removed: #{Enum.map_join(already_removing, ", ", &inspect(&1.name))}. This may lead to 'unknown child' errors.
       """)
     end

lib/membrane/core/parent/specification_parser.ex

@@ -27,7 +27,7 @@ defmodule Membrane.Core.Parent.SpecificationParser do
       |> Enum.map(fn
         %ChildrenSpec.Builder{links: links, children: children, status: :done} = builder ->
           if links == [] and children == [] do
-            Membrane.Logger.warn(
+            Membrane.Logger.warning(
               "The specification you have passed: #{builder} has no effect - it doesn't produce any children nor links."
             )
           end

lib/membrane/filter_aggregator.ex

@@ -36,7 +36,7 @@ defmodule Membrane.FilterAggregator do
   @impl true
   def handle_init(agg_ctx, %__MODULE__{filters: filter_specs}) do
     if filter_specs == [] do
-      Membrane.Logger.warn("No filters provided, #{inspect(__MODULE__)} will be a no-op")
+      Membrane.Logger.warning("No filters provided, #{inspect(__MODULE__)} will be a no-op")
     end
 
     states =

lib/membrane/logger.ex

@@ -79,7 +79,7 @@ defmodule Membrane.Logger do
     end
   end
 
-  Enum.map([:debug, :info, :warn, :error], fn method ->
+  Enum.map([:debug, :info, :warn, :warning, :error], fn method ->
     @doc """
     Wrapper around `Logger.#{method}/2` that adds Membrane prefix.