Merge pull request #1029 from laptou/master

allow `BorrowToSql` for non-static `Box<dyn ToSql>`
sfackler · May 7, 2023 · 22469d6 · 22469d6
2 parents 23d2524 + 066b466
commit 22469d6
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -55,7 +55,7 @@ jobs:
       - run: docker compose up -d
       - uses: sfackler/actions/rustup@master
         with:
-          version: 1.64.0
+          version: 1.65.0
       - run: echo "version=$(rustc --version)" >> $GITHUB_OUTPUT
         id: rust-version
       - uses: actions/cache@v3

diff --git a/postgres-types/src/lib.rs b/postgres-types/src/lib.rs
@@ -1184,17 +1184,17 @@ impl BorrowToSql for &dyn ToSql {
     }
 }
 
-impl sealed::Sealed for Box<dyn ToSql + Sync> {}
+impl<'a> sealed::Sealed for Box<dyn ToSql + Sync + 'a> {}
 
-impl BorrowToSql for Box<dyn ToSql + Sync> {
+impl<'a> BorrowToSql for Box<dyn ToSql + Sync + 'a> {
     #[inline]
     fn borrow_to_sql(&self) -> &dyn ToSql {
         self.as_ref()
     }
 }
 
-impl sealed::Sealed for Box<dyn ToSql + Sync + Send> {}
-impl BorrowToSql for Box<dyn ToSql + Sync + Send> {
+impl<'a> sealed::Sealed for Box<dyn ToSql + Sync + Send + 'a> {}
+impl<'a> BorrowToSql for Box<dyn ToSql + Sync + Send + 'a> {
     #[inline]
     fn borrow_to_sql(&self) -> &dyn ToSql {
         self.as_ref()