maxmind / MaxMind-DB-Reader-dotnet

.NET Reader for the MaxMind DB Database Format

Home Page:https://www.nuget.org/packages/MaxMind.Db

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Decoder.DecodePointer() unclear ternary op

Dmitry-Me opened this issue · comments

Decoder.DecodePointer() contains this code:

var b = pointerSize == 4 ? 0 : size & 0x7;

This code would be easier to read if it had composite parts of ternary op enclosed in braces:

var b = (pointerSize == 4) ? 0 : (size & 0x7);

I don't find the existing code particularly unclear, but if you submitted this as a PR, I'd be happy to merge it. I am going to close the issue.