Skip to content

Fixes for byte compiler & update travis#338

Open
jabranham wants to merge 6 commits intovermiculus:masterfrom
jabranham:fixes-for-byte-compiler
Open

Fixes for byte compiler & update travis#338
jabranham wants to merge 6 commits intovermiculus:masterfrom
jabranham:fixes-for-byte-compiler

Conversation

@jabranham
Copy link
Copy Markdown
Contributor

Various fixes so the byte compiler is satisfied, a few checkdoc fixes.

string-as-unibyte is obsolete in upcoming Emacs 26, Can you double-check that (encode-coding-string data 'no-conversion) is right?

@jabranham jabranham force-pushed the fixes-for-byte-compiler branch from 324bcb3 to 1a7608c Compare February 9, 2018 17:31
@jabranham
Copy link
Copy Markdown
Contributor Author

Rebased onto master @vermiculus

@vermiculus vermiculus self-assigned this Feb 9, 2018
@vermiculus
Copy link
Copy Markdown
Owner

Sorry I've been absent; I'll take a look at this by tomorrow night.

@jabranham
Copy link
Copy Markdown
Contributor Author

No problem, and no rush!

@jabranham jabranham force-pushed the fixes-for-byte-compiler branch from 1a7608c to 875b9b5 Compare February 16, 2018 15:45
@jabranham
Copy link
Copy Markdown
Contributor Author

Rebased, and fixed a few more things the byte compiler was complaining about

@jabranham
Copy link
Copy Markdown
Contributor Author

Pushed a commit to make travis error on compiler warnings

@jabranham jabranham changed the title Fixes for byte compiler Fixes for byte compiler & update travis Feb 16, 2018
@jabranham jabranham closed this Jul 24, 2018
@vermiculus
Copy link
Copy Markdown
Owner

Turns out I lied ☹️

Do you mind if I keep this open anyway?

@jabranham
Copy link
Copy Markdown
Contributor Author

Yes, sorry I didn't mean to close. Was looking at old lingering PRs and must've hit the wrong button.

@jabranham jabranham reopened this Jul 24, 2018
@vermiculus
Copy link
Copy Markdown
Owner

Is this something that's still valuable to merge after all this time? Happy to do so if so.

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