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

Time for a 1.30 release #32

Closed
fabiangreffrath opened this issue Oct 15, 2019 · 10 comments
Closed

Time for a 1.30 release #32

fabiangreffrath opened this issue Oct 15, 2019 · 10 comments

Comments

@fabiangreffrath
Copy link
Collaborator

I'd like to tag the current state of the code as a formal release.

@zvezdochiot @vividos @FoxesChen
You have contributed to the code lately. Is there anything missing in your opinion?

@zvezdochiot
Copy link

zvezdochiot commented Oct 15, 2019

A small remark.

I did not have time to verify the mp4 container after corrections. Good correctness detectors are the MPD and Sony Walkman player. In previous versions, I had to use FFMPEG to fix the container. Or used MP4Box for containerization (aac -> m4a).

@vividos
Copy link
Contributor

vividos commented Oct 15, 2019

There is a compiler error when compiling with VS 2019. In input.c the uint8_t data type is used, but the necessary <stdint.h> include is missing. Otherwise it compiles without errors.

@vividos
Copy link
Contributor

vividos commented Oct 15, 2019

with the latest commit faac compiles now without errors.

@fabiangreffrath
Copy link
Collaborator Author

@zvezdochiot Any clue what was wrong with the mp4 containers created by faac?

@zvezdochiot
Copy link

@fabiangreffrath say:

Any clue...

MPD just does not see them.
Sony Walkman plays only the first second.
I didn’t look under the hood.

@zvezdochiot
Copy link

@fabiangreffrath see:

Any clue...

Tested the git version on MPD. Sees.
Sony Walkman player is not at hand.

@fabiangreffrath
Copy link
Collaborator Author

Tested the git version on MPD. Sees.

I assume that's a good sign.

@zvezdochiot
Copy link

zvezdochiot commented Oct 16, 2019

@fabiangreffrath say:

I assume that's a good sign.

To understand what's under the hood, you need to take faac 1.28 and use https://github.com/madsen/vbindiff to look at the differences. But it takes time, I do not have it. (At 1.28 there was definitely a “bad” m4a container.)

@fabiangreffrath
Copy link
Collaborator Author

I am not going to do this, either. If you say the mp4 files produced by current git are properly recognized then I am fine with that.

@zvezdochiot
Copy link

"Comment and close":

https://github.com/knik0/faac/releases/tag/1_30

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

No branches or pull requests

3 participants