v18 -  Several "defaultt" typos found
#1
I noticed several "defaultt" typos in both Romanv's Kodistubs and xbmc's own code:

https://github.com/romanvm/Kodistubs/blo...i.py#L2229
https://github.com/xbmc/xbmc/search?q=de...q=defaultt

Is this deliberate?
Reply
#2
I'm sure it's intended cause "default"  is usually a reserved word hence "defaultt"  is used
Read/follow the forum rules.
For troubleshooting and bug reporting, read this first
Interested in seeing some YouTube videos about Kodi? Go here and subscribe
Reply
#3
I am certain this is going to break some older addons. Especially those which activate input dialogs with default strings.
Reply
#4
hmm.. how can something that hasn't been changed break anything?
Do not PM or e-mail Team-Kodi members directly asking for support.
Always read the Forum rules, Kodi online-manual, FAQ, Help and Search the forum before posting.
Reply
#5
default is a reserved keyword in C/C++ since forever so it is forbidden to use for custom names.
Raspberry PI 2 + LibreELEC 8 (Kodi 17)
Samsung Galaxy Tab A 10.1 + Kodi 17 for Android
Reply
#6
(2018-12-10, 00:32)ronie Wrote: hmm.. how can something that hasn't been changed break anything?
 I thought it was changed recently because I was using Kodistubs for quite a while and the positional argument WAS "default" instead of the double t. But Kodistubs has been kept up to date thanks to an automated script by romanvm, so that's why it pulled this keyword.
Reply
 
Thread Rating:
  • 0 Vote(s) - 0 Average



Logout Mark Read Team Forum Stats Members Help
Several "defaultt" typos found00