aboutsummaryrefslogtreecommitdiffstats
path: root/al
Commit message (Collapse)AuthorAgeFilesLines
...
* Don't inline some big functionsChris Robinson2023-09-161-40/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is very dumb. Template functions are implicitly marked inline according to C++, and contrary to popular belief, "inline" *does* influence the compiler's decision to inline a function. A function the compiler may not have decided to inline normally may be inlined anyway, or issue a warning if it still decides not to inline, if explicitly or implicitly marked as such (or does inline it as requested, but then decides to not inline different functions it normally would because of a cumulative code size increase or something). Furthermore, once a function becomes inline due to being a template function, there's no way to undo it. Marking an inline function "noinline" pushes the problem the other way, causing the compiler to not inline a function it may have decided was beneficial to inline. There's no way to declare a template function to be inlined based solely on compiler heuristics, it will always be influenced by the implicit "inline" or explicit "noinline". That's what's happening here. A number of functions had been combined into a smaller number of large-ish template functions to reduce code duplication and ease maintanence, causing them to be implicitly inline as a side-effect. GCC then manages to inline these larger functions as implicitly requested, but in doing so prevents other smaller functions (which are explicitly marked inline) from being inlined due to excessive code increase and issue a warning. The "noinline" is a heavy-handed method of un-pessimizing the optimization pass, on the assumption the compiler apparently doesn't actually want to inline the template functions, but does so because they're technically marked inline. There's no good option here until it gets acknowledged that inline does mean something beyond allowing multiple definitions, and that template (and other types of) function definitions sometimes (if not most often) want to allow multiple definitions but don't want an artificial/detrimental boost in inline prioritization. /rant
* Use duration_cast to convert nanoseconds to secondsChris Robinson2023-09-161-9/+14
|
* Replace another C string with string_viewChris Robinson2023-08-261-5/+4
|
* Simplify building a string messageChris Robinson2023-08-261-11/+3
|
* Use string_view in a couple more placesChris Robinson2023-08-254-38/+20
|
* Allow querying AL_EFFECTSLOT_EFFECTChris Robinson2023-08-252-5/+13
| | | | | | | This doesn't make much sense since the effect associated with the ID may be modified and not represent the effect being played, or was even deleted and isn't a valid effect ID, but Generic Software allows querying it so it should be queryable for compatibility if nothing else.
* Remove an unnecessary source fileChris Robinson2023-08-182-34/+11
|
* Use an array of bytes instead of ints for small valuesChris Robinson2023-08-151-4/+4
|
* Don't null check the context in the direct EAX functionsChris Robinson2023-08-131-16/+2
|
* Add missing includesChris Robinson2023-08-112-0/+3
|
* Add AL_EXT_debug functions to set/get object namesChris Robinson2023-08-1113-6/+188
|
* More properly print string_viewsChris Robinson2023-08-061-19/+7
|
* Fix if-comparison (#890)Johan Mattsson2023-07-301-1/+1
|
* Declare functions closer to where they're neededChris Robinson2023-07-267-131/+125
| | | | | This provides better code locality when calling implicit context functions that call their direct context variants.
* Don't hold the property lock when setting the event callbackChris Robinson2023-07-201-5/+3
|
* Add missing includeChris Robinson2023-07-101-0/+1
|
* Use a separate function to convert AL_EVENT_TYPE_ enumsChris Robinson2023-07-071-17/+18
|
* Fix some typos (#872)Dirk Stolle2023-07-043-5/+5
|
* Always use Windows' GUID definition on WindowsChris Robinson2023-06-111-8/+6
|
* Use cinttypes instead of inttypes.h in C++Chris Robinson2023-06-011-1/+1
|
* Rename threads.cpp/h to alsem.cpp/hChris Robinson2023-06-013-3/+0
|
* Avoid duplicate NULL checksChris Robinson2023-05-252-50/+44
|
* Add missing includeChris Robinson2023-05-241-0/+1
|
* Use a variant to hold EAX effect propertiesChris Robinson2023-05-2414-394/+399
|
* Be a bit more safe with type manglingChris Robinson2023-05-241-2/+2
|
* Increment an offset only when it's usedChris Robinson2023-05-241-1/+1
|
* Add a compat option to change the error value with no contextChris Robinson2023-05-241-1/+21
|
* Properly forward from alGetFilterivDirectChris Robinson2023-05-231-1/+1
|
* Use a variant to call the proper filter handler functionChris Robinson2023-05-232-258/+259
|
* Use a string_view for handling debug messagesChris Robinson2023-05-234-41/+41
|
* Use the macros to declare the indirect context thunksChris Robinson2023-05-232-42/+15
|
* Make the API functions noexceptChris Robinson2023-05-227-79/+47
| | | | | | | | | | | Only relevant for C++, but these functions can't throw as it's a C-based API. Letting the compiler know that helps improve code generation. Extension callbacks must also not let exceptions leave the callback, or else Bad Things can happen. The macro AL_DISABLE_NOEXCEPT may be defined before including the headers to not mark functions as noexcept, but this should only be done if the caller can't otherwise be fixed.
* Remove an unnecessary checkChris Robinson2023-05-151-3/+0
|
* Fix linkage definitionsChris Robinson2023-05-149-129/+125
|
* Don't check for a null context in direct functionsChris Robinson2023-05-1412-434/+125
|
* Implement direct functions for EFXChris Robinson2023-05-143-195/+194
|
* Implement direct functions for the debug APIChris Robinson2023-05-141-19/+40
|
* Implement direct functions for the listener and eventsChris Robinson2023-05-143-85/+92
|
* Implement direct functions for sourcesChris Robinson2023-05-141-277/+272
|
* Implement direct functions for buffersChris Robinson2023-05-142-181/+220
|
* Implement direct functions for extension queries and EAXChris Robinson2023-05-142-11/+23
|
* Implement direct functions for context stateChris Robinson2023-05-143-164/+157
|
* Avoid using al::vector unnecessarilyChris Robinson2023-05-126-18/+20
|
* Special-case setting a single buffer's modeChris Robinson2023-05-111-9/+46
| | | | To avoid extraneous allocations and extra work for a common case.
* Avoid counting buffers multiple times in EAXSetBufferModeChris Robinson2023-05-101-29/+28
|
* Mark some functions noexcept when possibleChris Robinson2023-05-102-15/+4
|
* Use a variant for AsyncEventChris Robinson2023-05-081-40/+54
|
* Send a debug message when disconnectingChris Robinson2023-05-081-5/+18
|
* Use an inline function instead of a macroChris Robinson2023-05-081-13/+15
|
* Store extension strings individuallyChris Robinson2023-05-082-11/+3
| | | | And dynamically build the full AL_EXTENSIONS string
generated by cgit v1.2.3 (git 2.39.1) at 2024-11-28 19:40:50 +0000