summaryrefslogtreecommitdiff
path: root/src (follow)
Commit message (Collapse)AuthorAgeFilesLines
* hle: Type check ResponseBuilder::Push arguments, and fix use in vi.cppGravatar comex2020-12-062-34/+37
| | | | | | | | | | | | - Add a type check so that calling Push with an invalid type produces a compile error rather than a linker error. - vi.cpp was calling Push with a variable of type `std::size_t`. There's no explicit overload for `size_t`, but there is one for `u64`, which on most platforms is the same type as `size_t`. On macOS, however, it isn't: both types are 64 bits, but `size_t` is `unsigned long` and `u64` is `unsigned long long`. Regardless, it makes more sense to explicitly use `u64` here instead of `size_t`.
* video_core: Adjust `NUM` macro to avoid Clang warningGravatar comex2020-12-063-3/+3
| | | | | | | | | | | | | | | | The previous definition was: #define NUM(field_name) (sizeof(Maxwell3D::Regs::field_name) / sizeof(u32)) In cases where `field_name` happens to refer to an array, Clang thinks `sizeof(an array value) / sizeof(a type)` is an instance of the idiom where `sizeof` is used to compute an array length. So it thinks the type in the denominator ought to be the array element type, and warns if it isn't, assuming this is a mistake. In reality, `NUM` is not used to get array lengths at all, so there is no mistake. Silence the warning by applying Clang's suggested workaround of parenthesizing the denominator.
* Merge pull request #5143 from comex/xx-users-sizeGravatar Rodrigo Locatti2020-12-061-1/+0
|\ | | | | yuzu_cmd: Remove 'users_size'
| * yuzu_cmd: Remove 'users_size'Gravatar comex2020-12-051-1/+0
| | | | | | | | | | | | | | | | | | | | | | Specifically: const auto size = sdl2_config->GetInteger("System", "users_size", 0); The variable is never used, producing a warning. I wondered if this ought to be assigning something to in `Settings`, but nothing else in the codebase ever mentions a setting called "users_size", so I guess it's safe to remove...
* | Merge pull request #5141 from comex/xx-true-falseGravatar Rodrigo Locatti2020-12-061-5/+7
|\ \ | | | | | | maxwell_dma: Rename RenderEnable::Mode::FALSE and TRUE to avoid name conflict
| * | maxwell_dma: Rename RenderEnable::Mode::FALSE and TRUE to avoid name conflictGravatar comex2020-12-051-5/+7
| |/ | | | | | | | | | | | | On Apple platforms, FALSE and TRUE are defined as macros by <mach/boolean.h>, which is included by various system headers. Note that there appear to be no actual users of the names to fix up.
* | Merge pull request #5140 from FearlessTobi/port-5577Gravatar bunnei2020-12-061-0/+1
|\ \ | | | | | | Port citra-emu/citra#5577: "Update cubeb and request a persistent stream session"
| * | Update cubeb and request a persistent stream sessionGravatar Vitor Kiguchi2020-12-051-0/+1
| | |
* | | Merge pull request #5132 from lioncash/xbyak-abiGravatar bunnei2020-12-061-10/+10
|\ \ \ | | | | | | | | xbyak_abi: Avoid implicit sign conversions
| * | | xbyak_abi: Shorten std::size_t to size_tGravatar Lioncash2020-12-051-8/+8
| | | | | | | | | | | | | | | | Makes for less reading.
| * | | xbyak_abi: Avoid implicit sign conversionsGravatar Lioncash2020-12-051-2/+2
| | | |
* | | | game_list_p: Resolve deprecated usage of QVariant operator<Gravatar Lioncash2020-12-051-1/+2
| |_|/ |/| | | | | | | | | | | This is designated as obsolete in Qt's docs (see: https://doc.qt.io/qt-5/qvariant-obsolete.html#operator-lt)
* | | video_core: Resolve more variable shadowing scenarios pt.2Gravatar Lioncash2020-12-0539-296/+305
| | | | | | | | | | | | | | | | | | | | | Migrates the video core code closer to enabling variable shadowing warnings as errors. This primarily sorts out shadowing occurrences within the Vulkan code.
* | | Merge pull request #5124 from lioncash/video-shadowGravatar bunnei2020-12-0542-206/+219
|\ \ \ | | | | | | | | video_core: Resolve more variable shadowing scenarios
| * | | video_core: Resolve more variable shadowing scenariosGravatar Lioncash2020-12-0442-206/+219
| | |/ | |/| | | | | | | | | | | | | Resolves variable shadowing scenarios up to the end of the OpenGL code to make it nicer to review. The rest will be resolved in a following commit.
* | | Merge pull request #5127 from FearlessTobi/port-5617Gravatar bunnei2020-12-041-3/+3
|\ \ \ | | | | | | | | Port citra-emu/citra#5617: "Fix telemetry-related exit crash from use-after-free"
| * | | Fix telemetry-related exit crash from use-after-freeGravatar FearlessTobi2020-12-051-3/+3
| |/ / | | | | | | | | | Co-Authored-By: xperia64 <xperia64@users.noreply.github.com>
* | / system_version: Update to 11.0.0Gravatar Chloe Marcec2020-12-051-6/+6
| |/ |/|
* | codec: Remove deprecated usage of AVCodecContext::refcounted_framesGravatar Lioncash2020-12-041-1/+0
|/ | | | | | | | | | | | | | | | This was only necessary for use with the avcodec_decode_video2/avcoded_decode_audio4 APIs which are also deprecated. Given we use avcodec_send_packet/avcodec_receive_frame, this isn't necessary, this is even indicated directly within the FFmpeg API changes document here on 2017-09-26: https://github.com/FFmpeg/FFmpeg/blob/master/doc/APIchanges#L410 This prevents our code from breaking whenever we update to a newer version of FFmpeg in the future if they ever decide to fully remove this API member.
* Merge pull request #5064 from lioncash/node-shadowGravatar bunnei2020-12-041-75/+77
|\ | | | | node: Eliminate variable shadowing
| * node: Mark member functions as [[nodiscard]] where applicableGravatar Lioncash2020-12-031-29/+29
| | | | | | | | Prevents logic bugs from accidentally ignoring the return value.
| * node: Eliminate variable shadowingGravatar Lioncash2020-12-031-47/+49
| |
* | Merge pull request #5061 from lioncash/pessimizingGravatar bunnei2020-12-032-11/+11
|\ \ | | | | | | vp9/vic: Resolve pessimizing moves
| * | vp9/vic: Resolve pessimizing movesGravatar Lioncash2020-12-032-11/+11
| |/ | | | | | | | | Removes the usage of moves that don't result in behavior different from a copy, or otherwise would prevent copy elision from occurring.
* | Merge pull request #4996 from bunnei/use-4jitsGravatar bunnei2020-12-0327-271/+214
|\ \ | | | | | | Kernel: Refactor to use 4-instances of Dynarmic & various cleanups and improvements
| * | kernel: scheduler: Minor cleanup to remove duplicated code.Gravatar bunnei2020-11-292-46/+14
| | |
| * | kernel: time_manager: Protect access with a mutex.Gravatar bunnei2020-11-292-1/+5
| | |
| * | common: fiber: Use VirtualBuffer for stack memory.Gravatar bunnei2020-11-291-2/+5
| | | | | | | | | | | | - This will be aligned by default, and helps memory usage.
| * | hle: kernel: thread: Remove unused "Running" state.Gravatar bunnei2020-11-293-21/+9
| | |
| * | core: arm: Implement InvalidateCacheRange for CPU cache invalidation.Gravatar bunnei2020-11-2912-16/+56
| | |
| * | hle: kernel: time_manager: Avoid a crash on process exit.Gravatar bunnei2020-11-291-1/+4
| | |
| * | hle: kernel: AddressArbiter: Remove unused code.Gravatar bunnei2020-11-292-9/+0
| | |
| * | hle: kernel: SynchronizationObject: Use atomic_bool for is_signaled.Gravatar bunnei2020-11-291-1/+2
| | |
| * | common: fiber: Use boost::context instead of native fibers on Windows.Gravatar bunnei2020-11-293-116/+9
| | |
| * | hle: kernel: multicore: Replace n-JITs impl. with 4 JITs.Gravatar bunnei2020-11-2915-72/+124
| | |
* | | mouse_poller: Remove unused includesGravatar Lioncash2020-12-031-3/+1
| | |
* | | mouse_input: Invert conditional in UpdateYuzuSettings()Gravatar Lioncash2020-12-031-4/+6
| | | | | | | | | | | | Allows the struct to be constructed in place.
* | | mouse_input: Remove two casts and amend some formattingGravatar Lioncash2020-12-031-11/+14
| | | | | | | | | | | | | | | Removes the use of two static casts and improves the readability of some vectors slightly.
* | | mouse_input: Resolve a -Wdocumentation warningGravatar Lioncash2020-12-031-1/+1
| | |
* | | mouse_input: Remove unused includesGravatar Lioncash2020-12-032-7/+3
| |/ |/|
* | Merge pull request #5000 from lioncash/audio-errorGravatar bunnei2020-12-0229-154/+163
|\ \ | | | | | | audio_core: Make shadowing and unused parameters errors
| * | audio_core: Make shadowing and unused parameters errorsGravatar Lioncash2020-12-0329-154/+163
| | | | | | | | | | | | Moves the audio code closer to enabling warnings as errors in general.
* | | Merge pull request #5002 from ameerj/nvdec-frameskipGravatar bunnei2020-12-0210-340/+234
|\ \ \ | | | | | | | | nvdec: Queue and display all decoded frames, cleanup decoders
| * | | Limit queue size to 10 framesGravatar ameerj2020-11-261-0/+4
| | | | | | | | | | | | | | | | Workaround for ZLA, which seems to decode and queue twice as many frames as it displays.
| * | | Address PR feedbackGravatar ameerj2020-11-264-32/+33
| | | | | | | | | | | | | | | | | | | | | | | | remove some redundant moves, make deleter match naming guidelines. Co-Authored-By: LC <712067+lioncash@users.noreply.github.com>
| * | | Queue decoded frames, cleanup decodersGravatar ameerj2020-11-2510-338/+227
| | | |
* | | | Merge pull request #4937 from german77/multiUDPGravatar bunnei2020-12-0110-267/+420
|\ \ \ \ | | | | | | | | | | InputCommon: Add multiple udp server support
| * | | | Add multiple udp server supportGravatar german2020-11-2510-267/+420
| | | | |
* | | | | Merge pull request #5047 from german77/MouseInputGravatar LC2020-12-011-6/+8
|\ \ \ \ \ | | | | | | | | | | | | InputCommon: Fix implicit conversion in mouse input
| * | | | | Fix implicit conversion in mouse inputGravatar german2020-11-301-6/+8
| | | | | |