Bond_009
5732e6188c
Fix some warnings
3 years ago
Bond_009
ea8f40e84a
More cleanup
3 years ago
Bond_009
1d19a5be61
Fix some warnings
...
down to 580
3 years ago
Patrick Barron
4ba9b6c305
Fix warnings in ScheduledTasks
3 years ago
Cody Robibero
cba07b1ca6
Remove more and more warnings
3 years ago
Bond_009
6f8ccab788
Move non-jellyfin extensions to separate project
3 years ago
Bond_009
0bc0601442
Fix some warnings
4 years ago
Bond_009
7e8428e588
Enable nullable reference types for Emby.Server.Implementations
4 years ago
BaronGreenback
2e98de9062
Code Clean up: Convert to null-coalescing operator ?? ( #5845 )
...
Co-authored-by: Cody Robibero <cody@robibe.ro>
Co-authored-by: Patrick Barron <18354464+barronpm@users.noreply.github.com>
4 years ago
BaronGreenback
f2c10471bf
Code Clean up: Use Pattern Matching ( #5838 )
...
Co-authored-by: Cody Robibero <cody@robibe.ro>
Co-authored-by: Patrick Barron <18354464+barronpm@users.noreply.github.com>
4 years ago
BaronGreenback
bc1cc2d04a
Remove unused using directives
4 years ago
cvium
42bcf171d9
Use sync Serialize when writing scheduled tasks to disk
4 years ago
Bond_009
a8ed753f6c
FxCop -> Net Analyzers (part 2)
4 years ago
dkanada
54a3ab15a3
Merge pull request #5005 from jellyfin/bytes
...
JsonSerializer deserialize from bytes where possible
4 years ago
Bond_009
1752423e52
Open FileStream with FileMode.Create instead of FileMode.OpenOrCreate
...
> The OpenWrite method opens a file if one already exists for the file path,
or creates a new file if one does not exist. For an existing file,
it does not append the new text to the existing text. Instead,
it overwrites the existing characters with the new characters.
If you overwrite a longer string
(such as "This is a test of the OpenWrite method") with a shorter string
(such as "Second run"), the file will contain a mix of the strings
("Second runtest of the OpenWrite method").
Ref: https://docs.microsoft.com/en-us/dotnet/api/system.io.file.openwrite?view=net-5.0#remarks
4 years ago
Bond_009
1ea2b200c0
JsonSerializer deserialize from bytes where possible
...
This is faster and uses way less memory
```
BenchmarkDotNet=v0.12.1, OS=fedora 32
Intel Core i7-6700HQ CPU 2.60GHz (Skylake), 1 CPU, 8 logical and 4 physical cores
.NET Core SDK=5.0.100
[Host] : .NET Core 5.0.0 (CoreCLR 5.0.20.51904, CoreFX 5.0.20.51904), X64 RyuJIT
DefaultJob : .NET Core 5.0.0 (CoreCLR 5.0.20.51904, CoreFX 5.0.20.51904), X64 RyuJIT
| Method | Mean | Error | StdDev | Gen 0 | Gen 1 | Gen 2 | Allocated |
|------- |---------:|--------:|--------:|--------:|------:|------:|----------:|
| Bytes | 158.4 us | 2.56 us | 2.14 us | 16.8457 | - | - | 52.08 KB |
| String | 172.8 us | 0.78 us | 0.70 us | 41.5039 | - | - | 127.82 KB |
| Custom | 155.5 us | 2.95 us | 2.76 us | 10.0098 | - | - | 31.27 KB |
```
4 years ago
David
3dec1fd6b2
Use UTF8 encoding and async correctly
4 years ago
David
21fd124bca
Code revie
4 years ago
David
043d045448
Put json serializer options in private field
4 years ago
David
e835dfb27d
Use sync string instead of file
4 years ago
David
2a574914ea
Use streams instead of strings
4 years ago
David
f38970cbd3
Remove xml docs
4 years ago
David
196388d607
Remove custom Json serializer from Emby.Server.Implementations
4 years ago
Bond_009
d4e568c8bf
Replace Task.WaitAll with Task.Wait
4 years ago
Bond_009
49569ca0a0
Use nameof where possible
4 years ago
Patrick Barron
9fa4fff15d
Merge branch 'master' into event-rewrite-1
...
# Conflicts:
# Emby.Dlna/Emby.Dlna.csproj
# Emby.Dlna/Eventing/DlnaEventManager.cs
# Emby.Dlna/Service/BaseService.cs
# Emby.Server.Implementations/ScheduledTasks/ScheduledTaskWorker.cs
# MediaBrowser.Controller/Subtitles/SubtitleDownloadEventArgs.cs
4 years ago
Bond_009
ab2147751f
Make MediaBrowser.MediaEncoding warnings free
4 years ago
Patrick Barron
ca1f15af19
Move GenericEventArgs to Jellyfin.Data.Events
4 years ago
Bond-009
9af6eda0b4
Merge pull request #3343 from telans/comment-stops
...
Add full stop at end of comments (SA1629)
4 years ago
telans
247f9c61e6
fix SA1513/SA1516
4 years ago
telans
9018f8d8be
Add full stop at end of comments (SA1629)
4 years ago
Vasily
40502a33e0
Merge pull request #3203 from Bond-009/warn18
...
Enable TreatWarningsAsErrors for Jellyfin.Server.Implementations in Release mode
5 years ago
Bond_009
4748105dce
Enable TreatWarningsAsErrors for Jellyfin.Server.Implementations in Release mode
5 years ago
Bond_009
30ce346f34
Enable nullabe reference types for MediaBrowser.Model
5 years ago
Bond-009
b477b3874e
Fix some warnings
5 years ago
Bond-009
6032f31aa6
Use CultureInvariant string conversion for Guids
5 years ago
Bond-009
d0f2b3a747
Merge branch 'master' into startuptasks
6 years ago
Erwin de Haan
d7c6d16250
Unwrapped CreateDirectory and DeleteDirectory
6 years ago
Erwin de Haan
3a831994f6
Unwrapped GetDirectoryName and DirectorySeperatorChar
6 years ago
Claus Vium
a05d803d4c
Fix crash when trying to deserialize a non-existing scheduled task
6 years ago
Bond_009
ffe79c8982
Check if file exists instead of catching exceptions
6 years ago
Bond_009
08b63a7c11
Remove unused function
...
RunStartupTasks never executes because "startuptasks.txt" is never
created
6 years ago
Bond_009
fd7f420af2
Remove remnants of system events
6 years ago
Erwin de Haan
65bd052f3e
ReSharper conform to 'var' settings
6 years ago
Erwin de Haan
0efc699e3d
Visual Studio Reformat: Emby.Server.Implementations Part S-S
6 years ago
Erwin de Haan
e9519de07d
Fixed removed start tag of XMLComment.
6 years ago
Erwin de Haan
846456b41e
Reformatted the files with notes.
6 years ago
Erwin de Haan
ec1f5dc317
Mayor code cleanup
...
Add Argument*Exceptions now use proper nameof operators.
Added exception messages to quite a few Argument*Exceptions.
Fixed rethorwing to be proper syntax.
Added a ton of null checkes. (This is only a start, there are about 500 places that need proper null handling)
Added some TODOs to log certain exceptions.
Fix sln again.
Fixed all AssemblyInfo's and added proper copyright (where I could find them)
We live in *current year*.
Fixed the use of braces.
Fixed a ton of properties, and made a fair amount of functions static that should be and can be static.
Made more Methods that should be static static.
You can now use static to find bad functions!
Removed unused variable. And added one more proper XML comment.
6 years ago
Bond_009
a997bb609f
Remove unnecessary using statements
6 years ago
Bond_009
391b48614d
Remove FireEventIfNotNull
...
It's a pretty useless "helper" class
6 years ago