Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings
This repository was archived by the owner on Aug 19, 2019. It is now read-only.

Add missing support for <s> tags for striketrough#26

Open
creynaud wants to merge 2 commits intodbowen:masterdbowen/NSAttributedString-DDHTML:masterfrom
creynaud:mastercreynaud/NSAttributedString-DDHTML:masterCopy head branch name to clipboard
Open

Add missing support for <s> tags for striketrough#26
creynaud wants to merge 2 commits intodbowen:masterdbowen/NSAttributedString-DDHTML:masterfrom
creynaud:mastercreynaud/NSAttributedString-DDHTML:masterCopy head branch name to clipboard

Conversation

@creynaud
Copy link

@creynaud creynaud commented Apr 18, 2017

This change is split into 2 commits:

  • the 1st one is to replace strncmp calls by strcmp ones ("s" was falling into the "strong" if case)
  • the actual addition of "s" support

<s> is also a tag that should be interpreted as strikethrough.

With strncmp, when a "s" tag is compared with "strong", it returns 0 because
only 1 character is compared -> use strcmp instead.
@creynaud
Copy link
Author

Hi,

Do you have any feedbacks on this pull request?

Thanks!
Claire

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Morty Proxy This is a proxified and sanitized view of the page, visit original site.