Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[EPG] Some EPG window cleanup #7556

Merged
merged 2 commits into from
Jul 21, 2015
Merged

[EPG] Some EPG window cleanup #7556

merged 2 commits into from
Jul 21, 2015

Conversation

ksooo
Copy link
Member

@ksooo ksooo commented Jul 20, 2015

Followup for #7509

@xhaggi mind taking a look

@ksooo ksooo added Type: Cleanup non-breaking change which removes non-working or unmaintained functionality v16 Jarvis labels Jul 20, 2015
@xhaggi
Copy link
Member

xhaggi commented Jul 20, 2015

@ksooo
Copy link
Member Author

ksooo commented Jul 20, 2015

https://github.com/ksooo/xbmc/blob/epg-window-fixes-2/xbmc/epg/GUIEPGGridContainer.cpp#L1411 cast could be removed?

Thx. Overlooked this one. Changed.

@xhaggi
Copy link
Member

xhaggi commented Jul 20, 2015

looking good so far

@ksooo
Copy link
Member Author

ksooo commented Jul 20, 2015

jenkins build this please

1 similar comment
@MartijnKaijser
Copy link
Member

jenkins build this please

@ksooo
Copy link
Member Author

ksooo commented Jul 21, 2015

Jenkins build errors are not related to this PR.

As "usual" addon build errors on the Pi, and since integration of the AudioDSP stuff errors with adsp.* addons.

@xhaggi
Copy link
Member

xhaggi commented Jul 21, 2015

shove it in

ksooo added a commit that referenced this pull request Jul 21, 2015
@ksooo ksooo merged commit 8f6dd20 into xbmc:master Jul 21, 2015
@hudokkow hudokkow added this to the J**** 16.0-alpha1 milestone Jul 21, 2015
@ksooo ksooo deleted the epg-window-fixes-2 branch July 21, 2015 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Cleanup non-breaking change which removes non-working or unmaintained functionality v16 Jarvis
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants