Yes I understand the "alignment" issue, but is insignificant compared to the other arguments (I think you agree)
As I said, it's a rather weak argument. Besides that, 99% has their tabstop at 4, which completely negates the argument imho.
merge requests for PLi's git
Started by Persian Prince, 16 Apr 2013 12:12
1748 replies to this topic
Re: merge requests for PLi's git #1441
-
- PLi® Core member
- 70,528 posts
+1,811
Excellent
Posted 30 April 2020 - 17:20
Currently in use: VU+ Duo 4K (2xFBC S2), VU+ Solo 4K (1xFBC S2), uClan Usytm 4K Ultimate (S2+T2), Octagon SF8008 (S2+T2), Zgemma H9.2H (S2+T2)
Due to my bad health, I will not be very active at times and may be slow to respond. I will not read the forum or PM on a regular basis.
Many answers to your question can be found in our new and improved wiki.
Re: merge requests for PLi's git #1442
-
- PLi® Core member
- 57,166 posts
+698
Excellent
Posted 30 April 2020 - 19:56
As far I know in python 3 the mix of tabs and spaces is not allowed anymore... so the obvious issue is already solved in python years ago... but somehow we are stuck on python 2.7 (while betacentauri did already made a python 3 branch which almost works more than a year ago).... there is so many things on ‘hold’.... (change to python 3 will break 3rd party plugins and more plugins)
WaveFrontier 28.2E | 23.5E | 19.2E | 16E | 13E | 10/9E | 7E | 5E | 1W | 4/5W | 15W
Re: merge requests for PLi's git #1443
-
- PLi® Contributor
- 2,499 posts
+64
Good
Posted 30 April 2020 - 19:57
Looks like we have a decision for Enigma2.
Tabs only.
Where we have a mix of spaces and tabs, convert space to tabs.
Agreed?
Sent from my Moto G (5S) using Forum Fiend v1.3.3.
Tabs only.
Where we have a mix of spaces and tabs, convert space to tabs.
Agreed?
Sent from my Moto G (5S) using Forum Fiend v1.3.3.
Re: merge requests for PLi's git #1444
-
- PLi® Core member
- 57,166 posts
+698
Excellent
Posted 30 April 2020 - 19:59
I think we go indeed to that direction.... but before pp pushes spaces to vix/openatv it is also smart to involve him.
WaveFrontier 28.2E | 23.5E | 19.2E | 16E | 13E | 10/9E | 7E | 5E | 1W | 4/5W | 15W
Re: merge requests for PLi's git #1445
-
- PLi® Core member
- 10,327 posts
+350
Excellent
Posted 30 April 2020 - 20:01
Why is this a dispute?
Let's close this topic about tabs.
class CIHelper: CI_ASSIGNMENT_LIST = None
CI_ASSIGNMENT_LIST(space)=(space)None
Nonsens...
Edited by Dimitrij, 30 April 2020 - 20:04.
GigaBlue UHD Quad 4K /Lunix3-4K/Duo 4K
Re: merge requests for PLi's git #1446
-
- PLi® Core member
- 57,166 posts
+698
Excellent
Posted 30 April 2020 - 20:09
That is pep.. this is to make it more readable... eg in cpp you should do the same.
This is also done in overal in e2.... so we should not change that.
This is also done in overal in e2.... so we should not change that.
Edited by littlesat, 30 April 2020 - 20:10.
WaveFrontier 28.2E | 23.5E | 19.2E | 16E | 13E | 10/9E | 7E | 5E | 1W | 4/5W | 15W
Re: merge requests for PLi's git #1447
-
- PLi® Core member
- 70,528 posts
+1,811
Excellent
Posted 30 April 2020 - 21:56
Agreed.
Currently in use: VU+ Duo 4K (2xFBC S2), VU+ Solo 4K (1xFBC S2), uClan Usytm 4K Ultimate (S2+T2), Octagon SF8008 (S2+T2), Zgemma H9.2H (S2+T2)
Due to my bad health, I will not be very active at times and may be slow to respond. I will not read the forum or PM on a regular basis.
Many answers to your question can be found in our new and improved wiki.
Re: merge requests for PLi's git #1448
-
- PLi® Core member
- 46,969 posts
+542
Excellent
Posted 1 May 2020 - 10:43
Looks like we have a decision for Enigma2.
Tabs only.
Where we have a mix of spaces and tabs, convert space to tabs.
Agreed?
We have been doing this for ages already.
* Wavefrontier T90 with 28E/23E/19E/13E via SCR switches 2 x 2 x 6 user bands
I don't read PM -> if you have something to ask or to report, do it in the forum so others can benefit. I don't take freelance jobs.
Ik lees geen PM -> als je iets te vragen of te melden hebt, doe het op het forum, zodat anderen er ook wat aan hebben.
Re: merge requests for PLi's git #1449
-
- PLi® Core member
- 46,969 posts
+542
Excellent
Posted 1 May 2020 - 10:45
Why is this a dispute?
Let's close this topic about tabs.class CIHelper: CI_ASSIGNMENT_LIST = NoneCI_ASSIGNMENT_LIST(space)=(space)None
Nonsens...
It's considered good practice in every language to insert spaces before and after symbols. Indeed for readability. In some languages it can even mean the difference between correct and incorrect interpretation by the interpreter or compiler in some situations.
* Wavefrontier T90 with 28E/23E/19E/13E via SCR switches 2 x 2 x 6 user bands
I don't read PM -> if you have something to ask or to report, do it in the forum so others can benefit. I don't take freelance jobs.
Ik lees geen PM -> als je iets te vragen of te melden hebt, doe het op het forum, zodat anderen er ook wat aan hebben.
Re: merge requests for PLi's git #1450
-
- Senior Member
- 1,982 posts
+247
Excellent
Re: merge requests for PLi's git #1451
-
- PLi® Core member
- 70,528 posts
+1,811
Excellent
Posted 30 July 2020 - 11:02
Thanks. We need to sort out this branding discussion asap so you don't have to do things like this.
Currently in use: VU+ Duo 4K (2xFBC S2), VU+ Solo 4K (1xFBC S2), uClan Usytm 4K Ultimate (S2+T2), Octagon SF8008 (S2+T2), Zgemma H9.2H (S2+T2)
Due to my bad health, I will not be very active at times and may be slow to respond. I will not read the forum or PM on a regular basis.
Many answers to your question can be found in our new and improved wiki.
Re: merge requests for PLi's git #1452
-
- Senior Member
- 1,982 posts
+247
Excellent
Posted 30 July 2020 - 11:09
Thanks. We need to sort out this branding discussion asap so you don't have to do things like this.
Yes I had to create another branch to keep the plugin compatible with PLi 
The new generation of Backup Suite doesn't have any lookuptable.txt or findkerneldevice as it's smarter.
If any of PLi team wanted to manage the master branch let me know to give you the write access.
You could check https://github.com/O...branding-module which is more compatible with PLi and how we're using it almost everywhere to reduce the checks.
Recipe: https://github.com/O...ion-branding.bb
P.S. you need so many variables but I could give you the needed PRs for all BSPs and your OE if you're up for it.
Edited by Persian Prince, 30 July 2020 - 11:11.
Open Vision sources: https://github.com/OpenVisionE2
Re: merge requests for PLi's git #1453
-
- Senior Member
- 391 posts
+8
Neutral
Posted 30 July 2020 - 11:31
Hi guys,
coming from 'true' OpenEmbedded I am now used to follow the discussed and agreed practices: https://www.openembe...wiki/Styleguide
So please at least for the metadata do follow this guide.
Ah, another ugly thing I see in *all* the OE-derivates is the lack of comments in the commits.
Here the guide: https://www.openembe...sage_Guidelines
I have also done some kernel work in the past and there indeed the bar is set very high!
In kernel however you use C and not a faulty languages like python with its ridicolous indenting risks:)
My 2 cents
A.A.
Re: merge requests for PLi's git #1454
-
- Senior Member
- 1,982 posts
+247
Excellent
Re: merge requests for PLi's git #1455
-
- Senior Member
- 1,982 posts
+247
Excellent
Re: merge requests for PLi's git #1456
-
- Senior Member
- 7,766 posts
+184
Excellent
Re: merge requests for PLi's git #1457
-
- PLi® Core member
- 57,166 posts
+698
Excellent
Posted 29 August 2020 - 12:46
Merged
WaveFrontier 28.2E | 23.5E | 19.2E | 16E | 13E | 10/9E | 7E | 5E | 1W | 4/5W | 15W
Re: merge requests for PLi's git #1458
-
- Senior Member
- 1,259 posts
+86
Good
Posted 5 November 2020 - 07:49
typo error
https://github.com/O...21d43a20758b378
........."vuduo4k", "vuduo4kse", "vuuno4k", "vuuno4kse", "vuultimo4k",)
Re: merge requests for PLi's git #1459
-
- PLi® Core member
- 57,166 posts
+698
Excellent
Posted 5 November 2020 - 07:52
Do you mean the comma at the end? It should not matter Technically.
WaveFrontier 28.2E | 23.5E | 19.2E | 16E | 13E | 10/9E | 7E | 5E | 1W | 4/5W | 15W
Re: merge requests for PLi's git #1460
-
- Senior Member
- 1,259 posts
+86
Good
Posted 5 November 2020 - 07:56
yes I mean that
5 user(s) are reading this topic
0 members, 2 guests, 0 anonymous users
-
Bing (3)
