annotate get_path.h @ 27518:e54c9b7eb0d8

Revert bad changes to SSA/ASS subtitle packet format The following commits are reverted partially or completely: "a valid ASS line contains 9 ',' before actual text" "demux_mkv: output correctly formated ASS packets" "libass: add a new ass_process_data() to process demuxed subtitle packets" These commits converted the internal representation of SSA/ASS subtitle packets from the format used by Matroska to a custom format where each packet has contents exactly matching one line in complete SSA script files. AFAIK no files natively use such a format for muxed subtitles. The stated reason for this change was to use a format that could in principle be muxed into a maximal number of containers. SSA subtitles do not have an implicit duration so both start time and duration or end time need to be specified explicitly; the new format moved timing information inside the codec packet data so it could be muxed without modification into containers that can represent only start time at the container level. However such a change is wrong from the viewpoint of program architecture. Timing information belongs to the demuxer level, but these commits moved not only the duration but also the authoritative value of the start time to inside the codec data. Additionally the new format lost the value of the Matroska ReadOrder field which is used by MPlayer. This commit changes the internal packet format back to that used by Matroska and makes the internal Matroska demuxer output that format again. Libavformat still outputs the "new" format; it could be converted back to the Matroska format in demux_lavf.c, but I'm not adding that code at least yet. The current lavf code has similar problems as the reverted code in MPlayer, and it also currently fails to provide any way to access the value of the ReadOrder field. I hope that the lavf side will be improved; if it isn't conversion can be added later. For now I'll make MPlayer default to the internal Matroska demuxer instead of the lavf one in a separate commit.
author uau
date Mon, 08 Sep 2008 21:26:22 +0000
parents 0f42fb42843c
children eda346733b8c
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
24242
76f5d8892c04 Clean up the way get_path is handled: Compile get_path.c to an object to link
diego
parents:
diff changeset
1 /*
76f5d8892c04 Clean up the way get_path is handled: Compile get_path.c to an object to link
diego
parents:
diff changeset
2 * Get path to config dir/file.
76f5d8892c04 Clean up the way get_path is handled: Compile get_path.c to an object to link
diego
parents:
diff changeset
3 *
76f5d8892c04 Clean up the way get_path is handled: Compile get_path.c to an object to link
diego
parents:
diff changeset
4 * This file is part of MPlayer.
76f5d8892c04 Clean up the way get_path is handled: Compile get_path.c to an object to link
diego
parents:
diff changeset
5 *
76f5d8892c04 Clean up the way get_path is handled: Compile get_path.c to an object to link
diego
parents:
diff changeset
6 * MPlayer is free software; you can redistribute it and/or modify
76f5d8892c04 Clean up the way get_path is handled: Compile get_path.c to an object to link
diego
parents:
diff changeset
7 * it under the terms of the GNU General Public License as published by
76f5d8892c04 Clean up the way get_path is handled: Compile get_path.c to an object to link
diego
parents:
diff changeset
8 * the Free Software Foundation; either version 2 of the License, or
76f5d8892c04 Clean up the way get_path is handled: Compile get_path.c to an object to link
diego
parents:
diff changeset
9 * (at your option) any later version.
76f5d8892c04 Clean up the way get_path is handled: Compile get_path.c to an object to link
diego
parents:
diff changeset
10 *
76f5d8892c04 Clean up the way get_path is handled: Compile get_path.c to an object to link
diego
parents:
diff changeset
11 * MPlayer is distributed in the hope that it will be useful,
76f5d8892c04 Clean up the way get_path is handled: Compile get_path.c to an object to link
diego
parents:
diff changeset
12 * but WITHOUT ANY WARRANTY; without even the implied warranty of
76f5d8892c04 Clean up the way get_path is handled: Compile get_path.c to an object to link
diego
parents:
diff changeset
13 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
76f5d8892c04 Clean up the way get_path is handled: Compile get_path.c to an object to link
diego
parents:
diff changeset
14 * GNU General Public License for more details.
76f5d8892c04 Clean up the way get_path is handled: Compile get_path.c to an object to link
diego
parents:
diff changeset
15 *
26743
0f42fb42843c Use standard license headers with standard formatting.
diego
parents: 26029
diff changeset
16 * You should have received a copy of the GNU General Public License along
0f42fb42843c Use standard license headers with standard formatting.
diego
parents: 26029
diff changeset
17 * with MPlayer; if not, write to the Free Software Foundation, Inc.,
0f42fb42843c Use standard license headers with standard formatting.
diego
parents: 26029
diff changeset
18 * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
24242
76f5d8892c04 Clean up the way get_path is handled: Compile get_path.c to an object to link
diego
parents:
diff changeset
19 */
76f5d8892c04 Clean up the way get_path is handled: Compile get_path.c to an object to link
diego
parents:
diff changeset
20
26029
4129c8cfa742 Add MPLAYER_ prefix to multiple inclusion guards.
diego
parents: 24242
diff changeset
21 #ifndef MPLAYER_GET_PATH_H
4129c8cfa742 Add MPLAYER_ prefix to multiple inclusion guards.
diego
parents: 24242
diff changeset
22 #define MPLAYER_GET_PATH_H
24242
76f5d8892c04 Clean up the way get_path is handled: Compile get_path.c to an object to link
diego
parents:
diff changeset
23
76f5d8892c04 Clean up the way get_path is handled: Compile get_path.c to an object to link
diego
parents:
diff changeset
24 char *get_path(const char *filename);
76f5d8892c04 Clean up the way get_path is handled: Compile get_path.c to an object to link
diego
parents:
diff changeset
25 void set_path_env();
76f5d8892c04 Clean up the way get_path is handled: Compile get_path.c to an object to link
diego
parents:
diff changeset
26
26029
4129c8cfa742 Add MPLAYER_ prefix to multiple inclusion guards.
diego
parents: 24242
diff changeset
27 #endif /* MPLAYER_GET_PATH_H */