Bläddra i källkod

LibAudio: Stop passing `Bytes` by reference

`Bytes` is very slim, so the memory and/or performance gains from
passing it by reference isn't that big, and it passing it by value is
more compatible with xvalues, which is handy for things like
`::try_create(buffer.bytes())`.
Tim Schumacher 2 år sedan
förälder
incheckning
3cf93d0dd2
2 ändrade filer med 3 tillägg och 3 borttagningar
  1. 1 1
      Userland/Libraries/LibAudio/Loader.cpp
  2. 2 2
      Userland/Libraries/LibAudio/Loader.h

+ 1 - 1
Userland/Libraries/LibAudio/Loader.cpp

@@ -56,7 +56,7 @@ Result<NonnullOwnPtr<LoaderPlugin>, LoaderError> Loader::try_create(StringView p
     return LoaderError { "No loader plugin available" };
 }
 
-Result<NonnullOwnPtr<LoaderPlugin>, LoaderError> Loader::try_create(Bytes& buffer)
+Result<NonnullOwnPtr<LoaderPlugin>, LoaderError> Loader::try_create(Bytes buffer)
 {
     NonnullOwnPtr<LoaderPlugin> plugin = adopt_own(*new WavLoaderPlugin(buffer));
     if (auto initstate = plugin->initialize(); !initstate.is_error())

+ 2 - 2
Userland/Libraries/LibAudio/Loader.h

@@ -72,7 +72,7 @@ protected:
 class Loader : public RefCounted<Loader> {
 public:
     static Result<NonnullRefPtr<Loader>, LoaderError> create(StringView path) { return adopt_ref(*new Loader(TRY(try_create(path)))); }
-    static Result<NonnullRefPtr<Loader>, LoaderError> create(Bytes& buffer) { return adopt_ref(*new Loader(TRY(try_create(buffer)))); }
+    static Result<NonnullRefPtr<Loader>, LoaderError> create(Bytes buffer) { return adopt_ref(*new Loader(TRY(try_create(buffer)))); }
 
     LoaderSamples get_more_samples(size_t max_samples_to_read_from_input = 128 * KiB) const { return m_plugin->get_more_samples(max_samples_to_read_from_input); }
 
@@ -89,7 +89,7 @@ public:
 
 private:
     static Result<NonnullOwnPtr<LoaderPlugin>, LoaderError> try_create(StringView path);
-    static Result<NonnullOwnPtr<LoaderPlugin>, LoaderError> try_create(Bytes& buffer);
+    static Result<NonnullOwnPtr<LoaderPlugin>, LoaderError> try_create(Bytes buffer);
 
     explicit Loader(NonnullOwnPtr<LoaderPlugin>);