Skip to content

Fixing character encoding conversion using Encode and other small fixes#4

Open
PhobosK wants to merge 2 commits intorobelix:masterfrom
PhobosK:master
Open

Fixing character encoding conversion using Encode and other small fixes#4
PhobosK wants to merge 2 commits intorobelix:masterfrom
PhobosK:master

Conversation

@PhobosK
Copy link

@PhobosK PhobosK commented Aug 1, 2015

Hi,
Here are the changes I propose:

  • I did change the way the encoding conversion is done. It doesn't use piconv by a system call anymore, since piconv is actually Perl Encode. I removed all system() calls since they are not needed and have problem with file naming (if unquoted, encoding etc). The logic now is that if the char encoding conversion fails, we at least will have a valid non-char-encoding-converted file... All this closes Error: function tmpnam undefined. #2 as well...
  • I did fix the subtitle formats naming mix-up, so this will close README: subtitle types mixed up #3 too...
  • Some spelling and formatting clean-ups
  • A version bump up to 0.5.6

If you like you may accept this pull request

Thanks

PhobosK added 2 commits August 1, 2015 02:14
[REMOVED] Usage of system calls and piconv
[FIXED] Some spelling mistakes
[FIXED] Minor fix for encoding conversion (closes robelix#2)
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.

README: subtitle types mixed up Error: function tmpnam undefined.

1 participant