summaryrefslogtreecommitdiff
path: root/src (follow)
Commit message (Collapse)AuthorAgeFilesLines
* services/sm: Amend error code constantsGravatar Lioncash2018-09-142-8/+6
| | | | | | | | Courtesy of @ogniK5377. This also moves them into the cpp file and limits the visibility to where they're directly used. It also gets rid of unused or duplicate error codes.
* ipc: minor fixGravatar Valentin Vanelslande2018-09-131-1/+1
|
* Use ARB_multi_bind for uniform buffers (#1287)Gravatar ReinUsesLisp2018-09-124-3/+27
| | | | | | * gl_rasterizer: use ARB_multi_bind for uniform buffers * address feedback
* Merge pull request #1298 from lioncash/viewGravatar bunnei2018-09-122-2/+4
|\ | | | | audio_core/sink_details: Change std::string parameter into std::string_view
| * audio_core/sink_details: Change std::string parameter into std::string_viewGravatar Lioncash2018-09-112-2/+4
| | | | | | | | | | The given string is only ever used for lookup and comparison, so we can just utilize a non-owning view to string data here
* | Merge pull request #1302 from lioncash/configGravatar bunnei2018-09-122-36/+74
|\ \ | | | | | | yuzu/configure_gamelist: Mark combo-box strings as translatable
| * | yuzu/configure_gamelist: Make combo box strings translatableGravatar Lioncash2018-09-122-21/+47
| | | | | | | | | | | | | | | | | | | | | | | | Given these are shown to the user, they should be translatable. While we're at it, also set up the dialog to automatically retranslate the dialog along with the combo boxes if it receives a LanguageChange event.
| * | yuzu/configure_gamelist: Use std::array instead of std::vector for ↵Gravatar Lioncash2018-09-121-6/+9
| | | | | | | | | | | | | | | | | | | | | translatable strings We don't need to use an allocating container for these, given we know the fixed amount of strings being used. This is just a waste of memory.
| * | yuzu/configure_gamelist: Move combo box initializtion to their own functionsGravatar Lioncash2018-09-122-23/+32
| | | | | | | | | | | | | | | | | | Keeps the individual initialization of the combo boxes logically separate. We also shouldn't be dumping this sort of thing in the constructor directly.
* | | Merge pull request #1163 from FearlessTobi/add-audio-stretchingGravatar bunnei2018-09-1218-49/+456
|\ \ \ | | | | | | | | audio_core: Add audio stretching support
| * | | audio_core: Flush stream when not playing anythingGravatar MerryMage2018-09-126-0/+23
| | | |
| * | | cubeb_sink: Downsample arbitrary number of channelsGravatar MerryMage2018-09-091-10/+9
| | | |
| * | | cubeb_sink: Perform audio stretchingGravatar MerryMage2018-09-083-24/+26
| | | |
| * | | audio_core: Add audio stretcherGravatar MerryMage2018-09-083-0/+101
| | | |
| * | | cubeb_sink: Hold last available value instead of writing zerosGravatar MerryMage2018-09-081-5/+15
| | | | | | | | | | | | | | | | This reduces clicking in output audio should we underrun.
| * | | cubeb_sink: Use RingBufferGravatar MerryMage2018-09-081-40/+26
| | | |
| * | | common: Implement a ring bufferGravatar MerryMage2018-09-084-0/+243
| | | |
| * | | Add audio stretching supportGravatar fearlessTobi2018-09-0812-0/+43
| | | |
* | | | gl_rasterizer_cache: B5G6R5U should use GL_RGB8 as an internal format.Gravatar bunnei2018-09-121-1/+1
| | | | | | | | | | | | | | | | - Fixes a regression with Sonic Mania with ARB_texture_storage.
* | | | Merge pull request #1297 from lioncash/plGravatar bunnei2018-09-122-66/+88
|\ \ \ \ | | | | | | | | | | pl_u: Eliminate mutable file-scope state
| * | | | pl_u: Eliminate mutable file-scope stateGravatar Lioncash2018-09-112-66/+88
| | |_|/ | |/| | | | | | | | | | | | | | Converts the PL_U internals to use the PImpl idiom and makes the state part of the Impl struct, eliminating mutable global/file state.
* | | | Merge pull request #1263 from FernandoS27/tex-modeGravatar bunnei2018-09-122-1/+43
|\ \ \ \ | | | | | | | | | | shader_decompiler: Implemented (Partially) Texture Processing Modes
| * | | | Implemented Texture Processing ModesGravatar FernandoS272018-09-122-1/+43
| |/ / /
* | | | Merge pull request #1303 from lioncash/errorGravatar bunnei2018-09-123-9/+11
|\ \ \ \ | | | | | | | | | | kernel/errors: Amend invalid thread priority and invalid processor ID error codes
| * | | | svc: Return ERR_INVALID_PROCESSOR_ID in CreateThread() if an invalid ↵Gravatar Lioncash2018-09-121-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | processor ID is given This is what the kernel does for an out-of-range processor ID.
| * | | | kernel/errors: Correct error codes for invalid thread priority and invalid ↵Gravatar Lioncash2018-09-123-7/+9
| | | | | | | | | | | | | | | | | | | | processor ID
* | | | | svc: Do nothing if svcOutputDebugString() is given a length of zeroGravatar Lioncash2018-09-121-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | While unlikely, it does avoid constructing a std::string and unnecessarily calling into the memory code if a game or executable decides to be really silly about their logging.
* | | | | svc: Correct parameter type for OutputDebugString()Gravatar Lioncash2018-09-122-3/+3
|/ / / / | | | | | | | | | | | | This should be a u64 to represent size.
* | | | Merge pull request #1296 from lioncash/prepoGravatar bunnei2018-09-112-39/+40
|\ \ \ \ | | | | | | | | | | service/prepo: Move class into the cpp file
| * | | | service/prepo: Move class into the cpp fileGravatar Lioncash2018-09-112-39/+40
| |/ / / | | | | | | | | | | | | | | | | | | | | This doesn't need to be exposed within the header and be kept in the translation unit, eliminating the need to include anything within the header.
* | | | Merge pull request #1301 from lioncash/qtGravatar bunnei2018-09-111-4/+4
|\ \ \ \ | |_|_|/ |/| | | game_list: Resolve variable shadowing within LoadCompatibilityList()
| * | | game_list: Resolve variable shadowing within LoadCompatibilityList()Gravatar Lioncash2018-09-111-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | "value" is already a used variable name within the outermost ranged-for loop, so this variable was shadowing the outer one. This isn't a bug, but it will get rid of a -Wshadow warning.
| * | | game_list: Use QJsonValueRef() within LoadCompatibilityList()Gravatar Lioncash2018-09-111-2/+2
| |/ / | | | | | | | | | This way, we aren't constructing unnecessary QJsonValue instances.
* | | Merge pull request #1300 from lioncash/audioGravatar bunnei2018-09-117-17/+34
|\ \ \ | | | | | | | | service/audio: Replace includes with forward declarations where applicable
| * | | service/audio: Replace includes with forward declarations where applicableGravatar Lioncash2018-09-117-17/+34
| |/ / | | | | | | | | | | | | A few headers were including other headers when a forward declaration can be used instead, allowing the include to be moved to the cpp file.
* | | Merge pull request #1278 from tech4me/bg-color-fixGravatar bunnei2018-09-116-0/+46
|\ \ \ | | | | | | | | Port Citra #4047 & #4052: add change background color support
| * | | Port Citra #4047 & #4052: add change background color supportGravatar tech4me2018-09-086-0/+46
| | | |
* | | | Merge pull request #1295 from bunnei/accurate-copiesGravatar bunnei2018-09-112-18/+12
|\ \ \ \ | | | | | | | | | | gl_rasterizer_cache: Improve accuracy of caching and copies.
| * | | | gl_rasterizer_cache: Always blit on recreate, regardless of format.Gravatar bunnei2018-09-111-6/+10
| | | | | | | | | | | | | | | | | | | | - Fixes several rendering issues with Super Mario Odyssey.
| * | | | gl_shader_cache: Remove cache_width/cache_height.Gravatar bunnei2018-09-112-12/+2
| | |/ / | |/| | | | | | | | | | | | | | - This was once an optimization, but we no longer need it with the cache reserve. - This is also inaccurate.
* | | | Merge pull request #1294 from degasus/optimizationsGravatar bunnei2018-09-113-11/+12
|\ \ \ \ | | | | | | | | | | gl_rasterizer: Use ARB_texture_storage.
| * | | | gl_rasterizer: Use ARB_texture_storage.Gravatar Markus Wick2018-09-113-11/+12
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | It allows us to use texture views and it reduces the overhead within the GPU driver. But it disallows us to reallocate the texture, but we don't do so anyways. In the end, it is the new way to allocate textures, so there is no need to use the old way.
* | | | Implemented LEA and PSETGravatar FernandoS272018-09-111-0/+91
| | | |
* | | | Implemented encodings for LEA and PSETGravatar FernandoS272018-09-111-0/+64
|/ / /
* | | Merge pull request #1291 from lioncash/defaultGravatar bunnei2018-09-11148-45/+291
|\ \ \ | | | | | | | | hle/service: Default constructors and destructors in the cpp file where applicable
| * | | hle/service: Default constructors and destructors in the cpp file where ↵Gravatar Lioncash2018-09-10148-45/+291
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | applicable When a destructor isn't defaulted into a cpp file, it can cause the use of forward declarations to seemingly fail to compile for non-obvious reasons. It also allows inlining of the construction/destruction logic all over the place where a constructor or destructor is invoked, which can lead to code bloat. This isn't so much a worry here, given the services won't be created and destroyed frequently. The cause of the above mentioned non-obvious errors can be demonstrated as follows: ------- Demonstrative example, if you know how the described error happens, skip forwards ------- Assume we have the following in the header, which we'll call "thing.h": \#include <memory> // Forward declaration. For example purposes, assume the definition // of Object is in some header named "object.h" class Object; class Thing { public: // assume no constructors or destructors are specified here, // or the constructors/destructors are defined as: // // Thing() = default; // ~Thing() = default; // // ... Some interface member functions would be defined here private: std::shared_ptr<Object> obj; }; If this header is included in a cpp file, (which we'll call "main.cpp"), this will result in a compilation error, because even though no destructor is specified, the destructor will still need to be generated by the compiler because std::shared_ptr's destructor is *not* trivial (in other words, it does something other than nothing), as std::shared_ptr's destructor needs to do two things: 1. Decrement the shared reference count of the object being pointed to, and if the reference count decrements to zero, 2. Free the Object instance's memory (aka deallocate the memory it's pointing to). And so the compiler generates the code for the destructor doing this inside main.cpp. Now, keep in mind, the Object forward declaration is not a complete type. All it does is tell the compiler "a type named Object exists" and allows us to use the name in certain situations to avoid a header dependency. So the compiler needs to generate destruction code for Object, but the compiler doesn't know *how* to destruct it. A forward declaration doesn't tell the compiler anything about Object's constructor or destructor. So, the compiler will issue an error in this case because it's undefined behavior to try and deallocate (or construct) an incomplete type and std::shared_ptr and std::unique_ptr make sure this isn't the case internally. Now, if we had defaulted the destructor in "thing.cpp", where we also include "object.h", this would never be an issue, as the destructor would only have its code generated in one place, and it would be in a place where the full class definition of Object would be visible to the compiler. ---------------------- End example ---------------------------- Given these service classes are more than certainly going to change in the future, this defaults the constructors and destructors into the relevant cpp files to make the construction and destruction of all of the services consistent and unlikely to run into cases where forward declarations are indirectly causing compilation errors. It also has the plus of avoiding the need to rebuild several services if destruction logic changes, since it would only be necessary to recompile the single cpp file.
* | | | Merge pull request #1292 from ogniK5377/renderdoc-fixGravatar bunnei2018-09-112-2/+9
|\ \ \ \ | | | | | | | | | | Fixed renderdoc input/output textures not working due to multiple render targets
| * | | | Fixed renderdoc input/output textures not working due to render targetsGravatar David Marcec2018-09-112-2/+9
| |/ / /
* / / / externals: Place font data within cpp filesGravatar Lioncash2018-09-111-6/+6
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This places the font data within cpp files, which mitigates the possibility of the font data being duplicated within the binary if it's referred to in more than one translation unit in the future. It also stores the data within a std::array, which is more flexible when it comes to operating with the standard library. Furthermore, it makes the data arrays const. This is what we want, as it allows the compiler to store the data within the read-only segment. As it is, having several large sections of mutable data like this just leaves spots in memory that we can accidentally write to (via accidental overruns, what have you) and actually have it work. This ensures the font data remains the same no matter what.
* | | Use open-source shared fonts if no dumped file is available (#1269)Gravatar Tobias2018-09-102-2/+26
| | | | | | | | | | | | | | | | | | * Add open-source shared fonts * Address review comments
HIC SVNT DRACONES