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

Delete adjacent spaces in edition/edition pattern #1311

Merged

Conversation

zuphilip
Copy link
Member

@zuphilip zuphilip commented Jan 1, 2015

Replacing the prefix/suffix adjacent spaces with group and one delimiter space, as described generally in #1301 . Search for this pattern here (70 hits in 70 files):

<[^/>]*"edition"[^/>]*suffix="[^"/>]* "[^/>]*/?>\s*\r\n\s*<[^/>]*"edition"[^/>]*prefix=" [^"/>]*"[^/>]*/?>

Every occurence here looks actually the same:

          <group>
            <text variable="edition" suffix=" "/>
            <text term="edition" prefix=" "/>
          </group>

which will then replaced by

          <group delimiter=" ">
            <text variable="edition"/>
            <text term="edition"/>
          </group>

Replacing the prefix/suffix adjacent spaces with group and one delimiter space, as described generally in citation-style-language#1301 . Search for this pattern here (70 hits in 70 files):

```
<[^/>]*"edition"[^/>]*suffix="[^"/>]* "[^/>]*/?>\s*\r\n\s*<[^/>]*"edition"[^/>]*prefix=" [^"/>]*"[^/>]*/?>
```

Every occurence here looks actually the same:

```
          <group>
            <text variable="edition" suffix=" "/>
            <text term="edition" prefix=" "/>
          </group>
```
which will then replaced by
```
          <group delimiter=" ">
            <text variable="edition"/>
            <text term="edition"/>
          </group>
```
adam3smith added a commit that referenced this pull request Jan 17, 2015
Delete adjacent spaces in edition/edition pattern
@adam3smith adam3smith merged commit d1af83e into citation-style-language:master Jan 17, 2015
@adam3smith
Copy link
Member

great, thanks. Sorry, these had slipped from my list of things to look at.

@zuphilip zuphilip deleted the adjacent-spacing-2 branch January 17, 2015 09:06
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