summaryrefslogtreecommitdiff
path: root/src (follow)
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #4574 from lioncash/const-fnGravatar bunnei2020-08-252-3/+3
|\ | | | | memory_manager: Mark IsGranularRange() as a const member function
| * memory_manager: Mark IsGranularRange() as a const member functionGravatar Lioncash2020-08-242-3/+3
| | | | | | | | This doesn't modify internal member state, so it can be marked as const.
* | Merge pull request #4563 from lioncash/rcacheGravatar bunnei2020-08-251-17/+16
|\ \ | | | | | | registered_cache: Make use of designated initializers
| * | registered_cache: Make use of ends_with for string suffix checkingGravatar Lioncash2020-08-231-2/+1
| | | | | | | | | | | | Simplifies code.
| * | registered_cache: Make use of designated initializersGravatar Lioncash2020-08-231-15/+15
| | | | | | | | | | | | Removes the need for comments to indicate the fields being assigned.
* | | Merge pull request #4548 from lioncash/colorGravatar bunnei2020-08-251-2/+2
|\ \ \ | | | | | | | | common/color: Migrate code over to the Common namespace
| * | | common/color: Migrate code over to the Common namespaceGravatar Lioncash2020-08-181-2/+2
| | | | | | | | | | | | | | | | | | | | No external code makes use of this header, so we can freely change the namespace.
* | | | Merge pull request #4542 from ReinUsesLisp/gpu-init-baseGravatar bunnei2020-08-2422-119/+172
|\ \ \ \ | | | | | | | | | | video_core: Initialize renderer with a GPU
| * | | | video_core: Initialize renderer with a GPUGravatar ReinUsesLisp2020-08-2222-119/+172
| | | | | | | | | | | | | | | | | | | | | | | | | Add an extra step in GPU initialization to be able to initialize render backends with a valid GPU instance.
* | | | | Merge pull request #4562 from lioncash/loopGravatar bunnei2020-08-241-16/+13
|\ \ \ \ \ | | | | | | | | | | | | cpu_manager: Make use of ranged for where applicable
| * | | | | cpu_manager: Make use of ranged for where applicableGravatar Lioncash2020-08-221-16/+13
| | |_|/ / | |/| | | | | | | | | | | | | We can simplify a few loops by making use of ranged for.
* | | | | gl_texture_cache: Take std::string by reference in DecorateViewName()Gravatar Lioncash2020-08-232-2/+2
| |_|_|/ |/| | | | | | | | | | | | | | | LabelGLObject takes a string_view, so we don't need to make copies of the std::string.
* | | | video_core/fence_manager: Remove unnecessary includesGravatar Lioncash2020-08-233-9/+4
| | | | | | | | | | | | | | | | | | | | Avoids pulling in unnecessary things that can cause rebuilds when they aren't required.
* | | | Merge pull request #4561 from lioncash/key-constexprGravatar bunnei2020-08-232-75/+82
|\ \ \ \ | | | | | | | | | | key_manager: Make data arrays constexpr
| * | | | key_manager: Make data arrays constexprGravatar Lioncash2020-08-222-75/+82
| |/ / / | | | | | | | | | | | | | | | | We can convert these maps into constexpr arrays to eliminate some runtime static constructors.
* | | | Merge pull request #4549 from lioncash/filesGravatar bunnei2020-08-231-32/+48
|\ \ \ \ | | | | | | | | | | vfs_real: Avoid redundant map lookups
| * | | | vfs_real: Resolve sign conversion warningsGravatar Lioncash2020-08-181-2/+2
| | | | |
| * | | | vfs_real: Avoid redundant map lookupsGravatar Lioncash2020-08-181-30/+46
| | |_|/ | |/| | | | | | | | | | | | | | Avoids some trivially avoidable map lookups by keeping the result of find operations around and querying them.
* | | | Merge pull request #4559 from lioncash/webresultGravatar bunnei2020-08-237-47/+41
|\ \ \ \ | | | | | | | | | | web_service: Move web_result.h into web_service
| * | | | web_service: Move web_result.h into web_serviceGravatar Lioncash2020-08-227-47/+41
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | This is the only place it's actively used. It's also more appropriate for web-related structures to be within the web service target. Especially given this one doesn't rely on anything in the common library.
* | | | Merge pull request #4560 from lioncash/convertGravatar bunnei2020-08-233-8/+6
|\ \ \ \ | | | | | | | | | | core_timing: Resolve sign conversion warning
| * | | | core_timing: Remove unused headerGravatar Lioncash2020-08-223-2/+2
| | | | |
| * | | | core_timing: Move clock initializer into constructor initializer listGravatar Lioncash2020-08-221-4/+2
| | | | | | | | | | | | | | | | | | | | Same behavior, minus unnecessary zeroing out of the pointer.
| * | | | core_timing: Resolve sign conversion warningGravatar Lioncash2020-08-221-2/+2
| |/ / / | | | | | | | | | | | | | | | | This constant is only ever assigned to downcount, which is a s64, not a u64.
* | | | Merge pull request #4521 from lioncash/optionalcacheGravatar bunnei2020-08-211-11/+12
|\ \ \ \ | | | | | | | | | | gl_shader_disk_cache: Make use of std::nullopt where applicable
| * | | | gl_shader_disk_cache: Make use of std::nullopt where applicableGravatar Lioncash2020-08-141-11/+12
| | | | | | | | | | | | | | | | | | | | | | | | | Allows the compiler to avoid unnecessarily zeroing out the internal buffer of std::optional on some implementations.
* | | | | Merge pull request #4541 from MerryMage/yoloGravatar bunnei2020-08-217-4/+108
|\ \ \ \ \ | | | | | | | | | | | | dynarmic: Add unsafe optimizations
| * | | | | dynarmic: Add unsafe optimizationsGravatar MerryMage2020-08-167-4/+108
| | | | | |
* | | | | | Merge pull request #4523 from lioncash/self-assignGravatar bunnei2020-08-211-1/+0
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | | macro-interpreter: Resolve -Wself-assign-field warning
| * | | | | macro-interpreter: Resolve -Wself-assign-field warningGravatar Lioncash2020-08-141-1/+0
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | This was assigning the field to itself, which is a no-op. The size doesn't change between its initial assignment and this one, so this is a safe change to make.
* | | | | Merge pull request #4546 from lioncash/telemetryGravatar bunnei2020-08-2013-35/+43
|\ \ \ \ \ | | | | | | | | | | | | common/telemetry: Migrate namespace into the Common namespace
| * | | | | common/telemetry: Migrate namespace into the Common namespaceGravatar Lioncash2020-08-1813-35/+43
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | Migrates the Telemetry namespace into the Common namespace to make the code consistent with the rest of our common code.
* | | | | Merge pull request #4547 from lioncash/header-conceptGravatar bunnei2020-08-191-2/+2
|\ \ \ \ \ | | | | | | | | | | | | common/concepts: Move <type_traits> include out of the Common namespace
| * | | | | common/concepts: Move <type_traits> include out of the Common namespaceGravatar Lioncash2020-08-181-2/+2
| |/ / / / | | | | | | | | | | | | | | | This is a compiler/linker error waiting to happen.
* | | | | Revert "common/time_zone: Simplify GetOsTimeZoneOffset()"Gravatar bunnei2020-08-191-5/+9
| | | | |
* | | | | Merge pull request #4539 from lioncash/discGravatar bunnei2020-08-182-3/+3
|\ \ \ \ \ | | | | | | | | | | | | common: Silence two discarded result warnings
| * | | | | common: Silence two discarded result warningsGravatar Lioncash2020-08-162-3/+3
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | These are intentionally discarded internally, since the rest of the public API allows querying success. We want all non-internal uses of these functions to be explicitly checked, so we can signify that we intentionally want to discard the return values here.
* | | | | Merge pull request #4522 from lioncash/vulk-copyGravatar bunnei2020-08-181-1/+1
|\ \ \ \ \ | |_|/ / / |/| | | | vulkan/wrapper: Avoid unnecessary copy in EnumerateInstanceExtensionProperties()
| * | | | vulkan/wrapper: Avoid unnecessary copy in EnumerateInstanceExtensionProperties()Gravatar Lioncash2020-08-141-1/+1
| | |/ / | |/| | | | | | | | | | | | | | Given this is implicitly creating a std::optional, we can move the vector into it.
* | | | Merge pull request #4381 from Morph1984/fix-open-folder-installed-titleGravatar bunnei2020-08-184-13/+24
|\ \ \ \ | | | | | | | | | | main: Fix Open Save/Mod Locations for installed titles
| * | | | main: Fallback to loader if no control nca is found with patch managerGravatar Morph2020-08-041-6/+17
| | | | | | | | | | | | | | | | | | | | In some rare instances, the patch manager is not able to find a control nca, fallback to the previous method of parsing a control nca through the loader if this occurs.
| * | | | main: Fix Open Save/Mod Locations for installed titlesGravatar Morph2020-08-044-12/+12
| | | | | | | | | | | | | | | | | | | | Previously NAND/SDMC installed titles would open device saves when they are supposed to be user saves. This is due to the control nca not being read and thus returns 0 for both GetDefaultNormalSaveSize() and GetDeviceSaveDataSize(). Fix this by utilizing the patch manager to read the control nca.
* | | | | Merge pull request #4532 from lioncash/object-nameGravatar bunnei2020-08-187-90/+74
|\ \ \ \ \ | | | | | | | | | | | | configuration_shared: Simplify name lookup in highlighting functions
| * | | | | configuration_shared: Simplify name lookup in highlighting functionsGravatar Lioncash2020-08-147-90/+74
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We can query the given object name directly from the widget itself. This removes any potential for forgetting to change the name if the widget gets renamed and makes the API much simpler (just pass in the widget, and not worry about its name).
* | | | | | Merge pull request #4535 from lioncash/fileutilGravatar bunnei2020-08-1740-547/+639
|\ \ \ \ \ \ | | | | | | | | | | | | | | common/fileutil: Convert namespace to Common::FS
| * | | | | | common/fileutil: Convert namespace to Common::FSGravatar Lioncash2020-08-1640-547/+639
| | |_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Migrates a remaining common file over to the Common namespace, making it consistent with the rest of common files. This also allows for high-traffic FS related code to alias the filesystem function namespace as namespace FS = Common::FS; for more concise typing.
* | | | | | Merge pull request #4537 from lioncash/tzGravatar bunnei2020-08-171-9/+5
|\ \ \ \ \ \ | | | | | | | | | | | | | | common/time_zone: Simplify GetOsTimeZoneOffset()
| * | | | | | common/time_zone: Simplify GetOsTimeZoneOffset()Gravatar Lioncash2020-08-161-9/+5
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We can simplify this function down into a single line with the use of fmt. A benefit with the fmt approach is that the fmt variant of localtime is thread-safe as well, making GetOsTimeZoneOffset() thread-safe as well.
* | | | | | Merge pull request #4540 from lioncash/tr3Gravatar bunnei2020-08-171-2/+2
|\ \ \ \ \ \ | | | | | | | | | | | | | | configure_hotkeys: Don't translate empty strings
| * | | | | | configure_hotkeys: Don't translate empty stringsGravatar Lioncash2020-08-161-2/+2
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | There's no need to translate an empty string. This just gives translators unnecessary work.
HIC SVNT DRACONES