Skip to content

Consider using the converters and JsonClassInfo from s_defaultOptions by cloning #32357

New issue
New issue
Closed
Closed
Consider using the converters and JsonClassInfo from s_defaultOptions by cloning#32357
Labels
area-System.Text.Jsonbacklog-cleanup-candidateAn inactive issue that has been marked for automated closure.tenet-performancePerformance related issue
Milestone
Future
@ahsonkhan

Description

@ahsonkhan
ahsonkhan
opened on Feb 15, 2020

runtime/src/libraries/System.Text.Json/src/System/Text/Json/Serialization/JsonSerializerOptions.cs

Lines 332 to 343 in d4b06b1

internal JsonClassInfo GetOrAddClass(Type classType)
{
_haveTypesBeenCreated = true;
// todo: for performance and reduced instances, consider using the converters and JsonClassInfo from s_defaultOptions by cloning (or reference directly if no changes).
if (!_classes.TryGetValue(classType, out JsonClassInfo? result))
{
result = _classes.GetOrAdd(classType, new JsonClassInfo(classType, this));
}
return result;
}

cc @steveharter

Metadata

Metadata

Assignees

No one assigned

    Labels

    area-System.Text.Jsonbacklog-cleanup-candidateAn inactive issue that has been marked for automated closure.tenet-performancePerformance related issue

    Type

    No type

    Projects

    No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions