Skip to content

Commit

Permalink
firebase: use bridging header and remove unnecessary header
Browse files Browse the repository at this point in the history
The `stdio.h` is a C header, which should not be required. Prefer using
the bridging header as that is available on newer toolchain releases.
  • Loading branch information
compnerd committed Feb 15, 2024
1 parent fc4ad08 commit 455998e
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions Sources/firebase/include/FirebaseCore.hh
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@

#include <firebase/util.h>

#include <stdio.h>
#include <swift/bridging>

namespace swift_firebase::swift_cxx_shims::firebase {

Expand All @@ -14,7 +14,7 @@ typedef void (*FutureCompletionType)(void*);
// Swift. We can ignore the `FutureBase` param as the Swift caller can just
// retain the Future as part of its closure.
template <class R>
class __attribute__((swift_attr("conforms_to:FirebaseCore.FutureProtocol")))
class SWIFT_CONFORMS_TO_PROTOCOL(FirebaseCore.FutureProtocol)
Future : public ::firebase::Future<R> {
public:
typedef R ResultType;
Expand Down

0 comments on commit 455998e

Please sign in to comment.