Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

LIB\NET45\MsgPack.XML TYPO #339

Open
wants to merge 20 commits into
base: master
Choose a base branch
from
Open

LIB\NET45\MsgPack.XML TYPO #339

wants to merge 20 commits into from

Conversation

Pekka-Kalevi
Copy link

@Pekka-Kalevi Pekka-Kalevi commented Apr 21, 2020

BUG REPORT

SOURCE FILE: MsgPack.Cli.0.6.8.nupkg

WINDOWS INSTALL PATH: C:\Program Files\PackageManagement\NuGet\Packages\MsgPack.Cli.0.6.8\lib\net45\MsgPack.XML

BUG COUNT: 1 of 870,899 bytes
CLASS: TYPO
SEVERITY: NONE

Line 1968 has a typo: 'othereise' should be 'otherwise'

1966
1967 <c>true</c> if the <paramref name="value"> is immutable collection;
1968 othereise, <c>false</c>.
1969

FIX:

Open file in PowerShell Administrator mode using install path.
Use Ctrl-G shortcut to get 'GoToLine Box'. Enter 1968. Correct 'e' to 'w'. Save the file.

Happy typos!

yfakariya and others added 20 commits December 1, 2013 14:00
(cherry picked from commit a6558fa)
* Collection traits detection does not care about source interface type itself
* Creating delegate in generated code emits wrong method reference due to above wrong traits.
Remove the references for DataContractAttribute and DataMemberAttribute
and re-implement with reflection-based attribute retrieving.
@yfakariya
Copy link
Member

Sorry for delayed response, and I am sorry for asking, please check followings:

  • Please include only typo fixes -- this PR looks contain too many fixes other than the typo.
  • It looks fix for 0.4 branch -- it is very old version. Revise that the target branch is right branch you want to fix.

Thank you for your contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants