Minor display issue in ServiceInfoList

  • Hi,


    There is a minor display issue in the file ServiceInfoList.py, located in /usr/lib/enigma2/python/Components. The code contains the following line:

    Code
    b = ("0x%0" + str(param) + "x (%dd)") % (to_unsigned(b), b)

    The formatting string (%dd) is incorrect. This results in an extra d appearing next to decimal values when displaying service information. For example, the display shows:

    Code
    0x0211 (529d)

    instead of the intended:

    Code
    0x0211 (529)

    Impact:

    • The only effect is cosmetic: decimal values are displayed with an extra d.

    • It affects technical numeric fields (PIDs).

    Resolution:

    The formatting should be corrected as follows:

    Code
    b = ("0x%0" + str(param) + "x (%d)") % (to_unsigned(b), b)

    This simple change removes the extra d and displays values correctly in hexadecimal and decimal format.


    There may be a future update to the Service Info screen to improve the display of long texts and support scrolling. :thinking_face:


    Best regards.

  • Andro1

    Changed the title of the thread from “Minor display issue in ServiceInfoListEntry” to “Minor display issue in ServiceInfoList”.
  • Well actually this wasn't an issue at all the d is explicitly added there to indicate it's a decimal value instead of hexadecimal.

  • Andro1

    Looks like a simple typo to me. The extra "d" is unnecessary, and removing it makes the values display clearly, just as in OpenATV.


    I also tried, with the help of AI, to modify the Service Information screen to display long texts and enable scrolling.


    So far, I can display long texts using "textwrap", but it doesn't look completely accurate.

    As for scrolling, the scrollbar appears, but pressing the arrow keys doesn’t work as expected!

  • Andro1 und Max11

    Habt ihr sonst keine Sorgen, wegen eines fragwürdigen "d" hier sowas überhaupt zu posten?

    Wenn ihr schon was fixen wollt, dann sucht euch echte Probleme und keine Sachen, die Jahrzehnte keinen interessiert haben.


    Sorry, aber das musste ich jetzt mal loswerden.

    Gruß Sven (aka Dreamy)


    DM920 mit unstable OE2.5 DP
    One + Two mit DP AIO - OE2.6

  • Reg Dich nicht auf, die sind stolz auf commits wo sie tab durch 4x blank ersetzen um den Code "aufzuräumen".


    Alles gut und kann auch nicht schaden, aber nach 20 Jahren jetzt mit sowas als "Tippfehler" vorbei zu kommen hat schon einen gewissen Unterhaltungswert.


    Oder die Kanalliste mit scrolltext zu versehenen - ein besserer Performance Killer würde mir auf die Schnelle gar nicht einfallen, etc.


    Ich könnten Ihnen auch den Patch geben wo ich mir den wert auch noch zusätzlich als xxxo also als Octal ausgeben ließ, das wäre dann ein tolles "Enhancement" :face_with_tongue:

    Edited once, last by gutemine ().

  • Of course, it’s good that we leave everything exactly as it is, as long as it "never bothered anyone" for 20 years.

    By that logic, anything that still works should obviously never be touched.


    Yes, it’s just a single character, and no one claimed it was a historic achievement.

    But don’t worry, I won’t waste anyone’s valuable time, it can be ignored just as easily as it has been for the last two decades.


    And about the octal output?

    A nice idea indeed, maybe one day we’ll add it as a feature "nobody cares about", just to stay consistent :face_with_tongue:

  • Sven H

    I get that this topic isn’t a priority for everyone.

    Still, it matters to some people, and that’s okay.

    Different interests are normal, and every discussion has its audience.

    Thanks for sharing your view, let’s just keep things respectful.