[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[elpa] externals/emms e1f2810f39 13/42: Use string instead of vector as
|
From: |
ELPA Syncer |
|
Subject: |
[elpa] externals/emms e1f2810f39 13/42: Use string instead of vector as Ogg page payload type |
|
Date: |
Wed, 1 Nov 2023 15:58:00 -0400 (EDT) |
branch: externals/emms
commit e1f2810f392bbea787aa9ed7553482b7d71e5934
Author: Petteri Hintsanen <petterih@iki.fi>
Commit: Petteri Hintsanen <petterih@iki.fi>
Use string instead of vector as Ogg page payload type
Strings use much less memory than vectors.
---
emms-info-ogg.el | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/emms-info-ogg.el b/emms-info-ogg.el
index ddec76b6fc..165cb8a1be 100644
--- a/emms-info-ogg.el
+++ b/emms-info-ogg.el
@@ -77,7 +77,7 @@ exhaustion in case of garbled or malicious inputs.")
(page-checksum u32r)
(page-segments u8)
(segment-table vec (page-segments))
- (payload vec (eval (seq-reduce #'+ last 0))))
+ (payload str (eval (seq-reduce #'+ last 0))))
"Ogg page structure specification.")
(defconst emms-info-ogg--magic-pattern
@@ -189,12 +189,12 @@ Only elementary streams are supported, that is, FILENAME
should
contain only a single logical stream. Note that this assumption
is not verified: with non-elementary streams packets from
different streams will be mixed together without an error."
- (let ((num-packets 0) (offset 0) (stream (vector)))
+ (let ((num-packets 0) (offset 0) (stream (string)))
(while (< num-packets packets)
(let ((page (emms-info-ogg--read-and-decode-page filename offset)))
(cl-incf num-packets (or (plist-get page :num-packets) 0))
(cl-incf offset (plist-get page :num-bytes))
- (setq stream (vconcat stream (plist-get page :stream)))
+ (setq stream (concat stream (plist-get page :stream)))
(when (> (length stream) emms-info-ogg--max-peek-size)
(error "Ogg payload is too large"))))
stream))
- [elpa] externals/emms 5a52c1b129 22/42: Remove dependency on cl-lib, (continued)
- [elpa] externals/emms 5a52c1b129 22/42: Remove dependency on cl-lib, ELPA Syncer, 2023/11/01
- [elpa] externals/emms 399dd78230 23/42: Use bindat-type in emms-info-ogg, ELPA Syncer, 2023/11/01
- [elpa] externals/emms e18b579e31 27/42: Use bindat-type in emms-info-mp3, ELPA Syncer, 2023/11/01
- [elpa] externals/emms f6bd021bc9 29/42: Compatibility code ert-resource-file, ELPA Syncer, 2023/11/01
- [elpa] externals/emms 9db19a5abd 31/42: Remove emms--use-bindat-type, ELPA Syncer, 2023/11/01
- [elpa] externals/emms c848c18727 33/42: Change to emms-info-native- prefix, ELPA Syncer, 2023/11/01
- [elpa] externals/emms 342c44103a 35/42: Fix multi-channel mapping in Opus identification header, ELPA Syncer, 2023/11/01
- [elpa] externals/emms b3c2f9cf09 39/42: Use uintr for little-endian unsigned integer fields, ELPA Syncer, 2023/11/01
- [elpa] externals/emms c96afb7687 40/42: Use eval-when-compile with subr-x, ELPA Syncer, 2023/11/01
- [elpa] externals/emms 2852a8f61b 10/42: Add tests for emms-info-native, ELPA Syncer, 2023/11/01
- [elpa] externals/emms e1f2810f39 13/42: Use string instead of vector as Ogg page payload type,
ELPA Syncer <=
- [elpa] externals/emms dd72caba90 37/42: Doc fixes, ELPA Syncer, 2023/11/01
- [elpa] externals/emms f594f7edac 15/42: Use strings instead of vectors for passing data, ELPA Syncer, 2023/11/01
- [elpa] externals/emms 2749fdb998 30/42: Allow empty metadata blocks, ELPA Syncer, 2023/11/01
- [elpa] externals/emms 52dac8ccc4 41/42: Remove most length limits from Vorbis bindat specs, ELPA Syncer, 2023/11/01
- [elpa] externals/emms 32fd570ed7 42/42: Merge branch 'info-native', ELPA Syncer, 2023/11/01