Skip to content

Conversation

@luckydrq
Copy link

I think it should use && here.

@hotoo
Copy link
Member

hotoo commented Sep 8, 2015

+1

@fengmk2 0.10 的覆盖率执行失败,重新执行下。

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

还是有点别扭,建议 if 分支顺序换下。

if (!charset || charset === 'utf-8' || charset === 'utf8') {
  content = content.toString();
} else {
  content = iconv.decode(content, charset);
}

@popomore
Copy link
Member

这个为了改什么

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.

3 participants