Rockbox.org home
Downloads
Release release
Dev builds dev builds
Extras extras
themes themes
Documentation
Manual manual
Wiki wiki
Device Status device status
Support
Forums forums
Mailing lists mailing lists
IRC IRC
Development
Bugs bugs
Patches patches
Dev Guide dev guide
Search



Donate

Rockbox Technical Forums


Login with username, password and session length
Home Help Search Staff List Login Register
News:

Welcome to the Rockbox Technical Forums!

+  Rockbox Technical Forums
|-+  Support and General Use
| |-+  User Interface and Voice
| | |-+  Viewing entire comments tag in MP3s
« previous next »
  • Print
Pages: 1 [2]

Author Topic: Viewing entire comments tag in MP3s  (Read 2067 times)

Offline Bilgus

  • Developer
  • Member
  • *
  • Posts: 876
Re: Viewing entire comments tag in MP3s
« Reply #15 on: March 24, 2021, 02:35:31 PM »
After a little more thought and not looking too closely at the current code :P
I think we could just use buflib to manage the static buffer and then you just need
to worry about which buffers are higher priority

but if you are going that far you might as well take it all the way and just make it buflib allocated memory too


I really dislike the idea of extend tags for comments
 
Logged

Offline Frankenpod

  • Member
  • *
  • Posts: 641
Re: Viewing entire comments tag in MP3s
« Reply #16 on: March 28, 2021, 02:34:20 PM »
Quote from: Bilgus on March 24, 2021, 09:15:16 AM
where does it end 1000 char comments or 1000 megs?

IDK after that maybe make a chunking display in the theme engine?

or just call it good enough for most cases

My impression is most podcasts have 300-1000 character comments.  Maybe there's a way to write a script to look at a lot of them and find what the range/average is (but at the moment I can only copy the comments to a text editor and do a character count, which is time-consuming.  Typical BBC podcasts I've looked at seem to have comments 700-1500 characters long).  The current situation thanks to Chris_S seems like an acceptable compromise, though I don't know how tight memory is for targets with 32 or 64mb and whether there is a downside to going any higher.

  Not sure what a 'chunking display' means, unless it means chopping longer comment fields up into substrings?  I don't understand the details of how tag reading works in Rockbox, but would that still not need a longer string capacity to get access to the full comment field in the first place?

When you say you dislike 'extend tags for comments', do you mean 'extending tags for comments' or 'extended tags for comments', and do you mean 'within rockbox' or just that you dislike podcasters using comment tags in the way they seem to do?
« Last Edit: March 28, 2021, 02:37:08 PM by Frankenpod »
Logged

Offline Bilgus

  • Developer
  • Member
  • *
  • Posts: 876
Re: Viewing entire comments tag in MP3s
« Reply #17 on: March 29, 2021, 02:37:35 PM »
 I mean extended tags like having Cmt0 Cmt1 Cmt2
I feel like that is a burden all the way around when it should just work

The other issue I have with just increasing the buffer is that its static and gone forever when it could be either managed static or totally allocated from the start
..its not a longterm solution
Logged

  • Print
Pages: 1 [2]
« previous next »
+  Rockbox Technical Forums
|-+  Support and General Use
| |-+  User Interface and Voice
| | |-+  Viewing entire comments tag in MP3s
 

  • SMF 2.0.19 | SMF © 2021, Simple Machines
  • Rockbox Privacy Policy
  • XHTML
  • RSS
  • WAP2

Page created in 0.023 seconds with 18 queries.