1. Aug 07, 2021
    • Eric Cousineau's avatar
      test_eval: Show example of working closure (#2743) · 6ac8efe5
      Eric Cousineau authored
      
      
      * test_eval: Show example of working closure
      
      * Extend test_eval_closure with weirder examples of closures for py::eval
      
      Co-authored-by: default avatarYannick Jadoul <yannick.jadoul@belgacom.net>
      Co-authored-by: default avatarAaron Gokaslan <skylion.aaron@gmail.com>
      6ac8efe5
    • Ralf W. Grosse-Kunstleve's avatar
      Removing GCC -Wunused-but-set-parameter from pragma block at the top of pybind11.h (#3164) · af700733
      Ralf W. Grosse-Kunstleve authored
      
      
      * Cleanup triggered by work on pragma for GCC -Wunused-but-set-parameter.
      
      * Backing out changes to eigen.h (to be worked on later).
      
      * Adding PYBIND11_WORKAROUND_INCORRECT_GCC_UNUSED_BUT_SET_PARAMETER in type_caster_base.h (apparently needed only for older GCCs).
      
      * Apparently older compilers need a simpler overload for silence_unused_warnings().
      
      * clang C++11 compatibility: removing constexpr
      
      * Special case for MSVC 2017: `constexpr void` return
      
      * Trying again without the silence_unused_warnings(const int *) overload.
      
      * Separate macros for ALL_GCC, OLD_GCC_UNUSED_BUT_SET_PARAMETER
      
      * Changing to __GNUC__ <= 2 (turning off)
      
      * Refined condition for PYBIND11_WORKAROUND_INCORRECT_OLD_GCC_UNUSED_BUT_SET_PARAMETER.
      
      * Quick experiment trying out suggestion by @henryiii
      
      * Introducing macro: PYBIND11_INT_ARRAY_WORKING_AROUND_MSVC_CLANG_ISSUES
      
      * Trying henryiii@ (void) expander idea.
      
      * fix: apply simpler expression with fewer workarounds
      
      * Purging new-but-already-obsoleted  macro, made possible by @henryiii's commit.
      
      * Renaming `ALL_GCC` macro back to just `GCC` (because there is no `OLD` anymore, luckily).
      
      * [actions skip] Adding "All GCC versions" to comment, to be clear about it.
      
      Co-authored-by: default avatarHenry Schreiner <henryschreineriii@gmail.com>
      af700733
    • Aaron Gokaslan's avatar
      maint(clang-tidy): Bugprone enable checks (#3166) · 3893f37b
      Aaron Gokaslan authored
      * Enable bugprone checks
      
      * Reset delta and massage config
      
      * Start to apply bugprone fixes
      
      * try to fix minor bug
      
      * Fix later
      
      * Fix perfect forwarding bugprone
      
      * Remove nolint
      
      * undo constructor delete
      
      * Fix bugprone-perfect-forwarding again
      
      * Remove TODO
      
      * Add another nolint for bugprone-exception-escape in scoped interpreter
      
      * Fix remaining bugprone errors
      
      * Properly apply bugprone-macro-parantheses
      
      * Redo formatting and remove bugprone nolint
      
      * Add coment and revert more whitespace changes
      
      * Fix typo
      
      * Fix parsing bug
      
      * Add back comma
      
      * Fix clang-tidy issue
      
      * Apply remaining clang-tidy fixes
      3893f37b
    • Henry Schreiner's avatar
      Revert "fix: apply simpler expression with fewer workarounds" · 089328f7
      Henry Schreiner authored
      This reverts commit 1fafd1b4.
      089328f7
    • Henry Schreiner's avatar
      1fafd1b4
  2. Aug 04, 2021
  3. Aug 03, 2021
  4. Jul 31, 2021
  5. Jul 30, 2021
  6. Jul 29, 2021
  7. Jul 28, 2021
  8. Jul 27, 2021
  9. Jul 22, 2021
  10. Jul 21, 2021
    • jesse-sony's avatar
      Feature/local exception translator (#2650) · d65edfb0
      jesse-sony authored
      * Create a module_internals struct
      
      Since we now have two things that are going to be module local, it felt
      correct to add a struct to manage them.
      
      * Add local exception translators
      
      These are added via the  register_local_exception_translator function
      and are then applied before the global translators
      
      * Add unit tests to show the local exception translator works
      
      * Fix a bug in the unit test with the string value of KeyError
      
      * Fix a formatting issue
      
      * Rename registered_local_types_cpp()
      
      Rename it to get_registered_local_types_cpp() to disambiguate from the
      new member of module_internals
      
      * Add additional comments to new local exception code path
      
      * Add a register_local_exception function
      
      * Add additional unit tests for register_local_exception
      
      * Use get_local_internals like get_internals
      
      * Update documentation for new local exception feature
      
      * Add back a missing space
      
      * Clean-up some issues in the docs
      
      * Remove the code duplication when translating exceptions
      
      Separated out the exception processing into a standalone function in the
      details namespace.
      
      Clean-up some comments as per PR notes as well
      
      * Remove the code duplication in register_exception
      
      * Cleanup some formatting things caught by clang-format
      
      * Remove the templates from exception translators
      
      But I added a using declaration to alias the type.
      
      * Remove the extra local from local_internals variable names
      
      * Add an extra explanatory comment to local_internals
      
      * Fix a typo in the code
      d65edfb0
  11. Jul 20, 2021
  12. Jul 17, 2021
  13. Jul 16, 2021