Merge pull request #3562 from Karlson2k/vfs_update_small_02

[vfs] CFile: remove 'virtual' from destructor

Merge pull request #3709 from bavison/cache_utf8_text

Add caching of text labels at the UTF8 level

Merge pull request #3785 from ronie/confluence-rightbuttons

[Confluence] move buttons to the right side, improve navigation

Add caching of text labels at the UTF8 level

Previously, CGUITextLayout::Update converted its string argument from UTF8 to
UTF16, then passed it to CGUITextLayout::UpdateW, where it was compared
against a cached copy of the string (in its UTF16 form). However, the
conversion itself was taking up a reasonable chunk of CPU time.

This patch adds a similar cache of the string in its UTF8 form to the Update
method, and splits the non-cache part of UpdateW into a separate function, so
we can bypass it the case where we've tested it against the UTF8 cache.

I have measured the effect while the Videos window of the default skin was
open (but idle) on a Raspberry Pi, and this reduced the CPU usage by 2.9%
from 39.1% to 36.2%:

          Before          After
          Mean   StdDev   Mean   StdDev  Confidence  Change
IdleCPU%  39.1   0.9      36.2   0.5     100.0%      +8.1%

Change the string comparison in CGUITextLayout::UpdateW to be case-sensitive

It seems likely that it was only case-insensitive due to the the default
seting of CStdStr::Equals. It's unlikely that you'd ever want to update a
string with another which is the same except for the case of one or more
characters, but if you did, I doubt you'd want to use the cached version if
so. And you're slowing down the comparison in all other cases to achieve that
dubious behaviour.

Merge pull request #3772 from nigeljonez/nfsfix-14727

[nfs] Fix syncing issue from Trac #14727

CharsetConverter::ToUtf8: add ability to fail on bad character

StringUtils::Trim: add overload with trim chars parameter

StringUtils::TrimLeft/TrimRight: pass trimming chars as constant character strings

Merge pull request #3777 from Montellese/season_listing_fix

videolibrary: move logic for tvshow/season flattening from filesystem to GUI (fixes #14723)

Merge pull request #3618 from siriuzwhite/avformat-hls-fix

fixed bug where ffmpeg doesn't keep custom http headers when playing hls stream

ffmpeg: add patch for backport: fixed not keeping custom http headers

[backport] fixed ffmpeg not keep custom http headers when playing hls stream

[nfs] Fix library syncing (trac #14727) for edge-cases resulting from less-common NFS export configurations partially resulting from recent CStdString conversion.

Squashed commit of the following:

commit 1cd13db9822a5a609c232a614e67c361ca260f2d
Author: Nigel Jones <nigel@nigelj.com>
Date:   Sun Dec 8 02:47:08 2013 +1300

    [nfs] Recommendation from PR 3772 by jmarshallnz for code cleanup implemented avoiding the substr routine.

    exportPath is only set if a valid export is found.

commit be9d0b0aa42b4710a5005824c1d04f1a69120cbc
Merge: a43d406 d32ff6e
Author: Nigel Jones <nigel@nigelj.com>
Date:   Sat Dec 7 03:09:32 2013 -0800

    Merge pull request #1 from Memphiz/nfsfix-14727

    [nfs] - handle special case when "/" is exported via nfs - we need to al...

commit d32ff6e89bde24909fbf7e97acf0bfa5a8ae5ac7
Author: Memphiz <memphis@machzwo.de>
Date:   Fri Dec 6 23:35:48 2013 +0100

    [nfs] - handle special case when "/" is exported via nfs - we need to allow empty pathes here when accessing nfs://ip/ (path is empty here - export is "/").

commit a43d40673887965041939181dab8dcd077049c92
Author: Nigel Jones <nigel@nigelj.com>
Date:   Sat Dec 7 00:27:49 2013 +1300

    [nfs] - Ensure that the right export is picked if multiple overlapping names may match with StartsWith.

commit 61fb6f567f0a229c1869bbb76524706914574ba9
Author: Nigel Jones <nigel@nigelj.com>
Date:   Fri Dec 6 23:44:03 2013 +1300

    [nfs] replacement fix for dd4dd2e91bc083a23fe7b46ae4c291c6672d32e0 (issue #14727).

    If the exportPath is the same as the path is then bypass the substr methods, and return the correct relativePath.

Merge pull request #3776 from Memphiz/nfsunittest

[tests] - add unit test for nfs

[nfs] - add unit tests for verifying proper splitting of nfs urls into export and relative path

Merge pull request #3744 from FernetMenta/aefixes

Aefixes

[Confluence] move buttons to the right side, improve navigation

Merge pull request #3784 from wsoltys/peripherals

[peripherals] fixed a ambiguous variable and replaced .size() by .empty()

Merge pull request #3787 from jimfcarroll/fix-doxygen-markdown

Minor Codegenerator Doxygen Fix

Fix codegenerator to handle the <emphasis> doxygen generated by just putting back the markdown.

WASAPI: handle obsolete 7.1 wide layout

WASAPI: try other channels layouts on init if requested one is not supported

ActiveAE: fix incorrect return values of GetSpace

ActiveAE: limit buffertime to 80ms

AESinkAlsa: Use 51Wide and 71Wide Maps when we need to do so

Merge pull request #3782 from wsoltys/reference

changed: pass CStreamDetailSubtitle as reference

[Confluence] cosmetics - font colors

Merge pull request #3587 from Nachtzuster/ma4_replaygain

fix: Use replaygain tags for .m4a type family files too, closes Ticket #13948

[peripherals] fixed a ambiguous variable and replaced .size() by .empty()

removed double assignment in EpgSearchFilter::Reset()

Merge pull request #3781 from afedchin/dxva-hd-fix-warn

[dxva-hd] remove mistakenly redefined members

[dxva-hd] remove mistakenly redefined members after 04bb49cc4475778a82da86673d3f3486193c5354

changed: pass CStreamDetailSubtitle as reference

[WIN32] changed: log a more meaningful message if DXVAHD fails because the device doesn't support it.

ActiveAE: let sink do channel mapping for 7.1

AE: allow channel BC to be mixed into BL/BR os SL/SR

AE: fix hidden setting stereo upmix if spdif is selected

[cosmetic] cleanup copyright headers

set Touched revision back after 35e2b91257eb6c1dc614589672f4324fdcc234da

Merge pull request #3751 from elupus/gles

Minor gles renderer cleanup and fixes

[WIN32] fixed: wrong member initialization

Merge pull request #3589 from afedchin/dxva-hd

[dxva] added dxva-hd renderer as alternative native dxva video processing.

[win32] update PCRE depend package
.pdb files now included in package

Merge pull request #3678 from Karlson2k/cleanup_smb_01

SMB: Cleanup

get rid of two cast warnings

videolibrary: move logic for tvshow/season flattening from filesystem to GUI (fixes #14723)

Merge pull request #3775 from jmarshallnz/more_caching_of_bools_in_listitems

More caching of bools in listitems

[nfs] - change splitUrlIntoExportAndPath so that it can be used with external exportlist - preparation for adding a unit test

[test] - fix compile of TestCharsetConverter after 8f830c1b9a4446f9b68f0dbd693fcda2788eff26

[tests] - fix compile of TestCPUInfo after string changes

Merge pull request #3762 from xhaggi/confluence-pvr

[confluence] improve pvr timer and search view

[confluence] improving skin code for search view

[confluence] improving skin code for timer view

[info] Have the infomanager invalidate infobools so that classes that hold them can call the infobool directly rather than going via the infomanager

[info] Add selective caching of infobools in list items

As noted in a TODO comment attached to CGUIInfoManager::GetBoolValue, caching
of infobools (both expression and single bools) is disabled whenever a
CGUIListItem pointer is provided. This is true whenever visibility is
evaluated for a control which is part of a list item (which is responsible
for a large portion of the controls in many windows), which obviously hurts
cache efficiency. However, many of the bools (including the leaf nodes of
expression infobools) aren't actually dependent upon which list item they're
attached to.

This patch adds a flag to each InfoBool object to indicate whether its value
depends upon the list item; if this isn't set, then the cache is re-enabled
even if a CGUIListItem pointer is provided. The flag is calculated at parse
time because it's quite tricky to evaluate - it applies to bools in any of
the following categories:
* a statically definied boolean condition in the range LISTITEM_START to
  LISTITEM_END - for example, "ListItem.IsResumable"
* a (dynamically allocated) multi-info bool, that is to say, the result of a
  string or integer operation on a list item attribute - for example
  "substring(Listitem.mpaa,Rated R)"
* an expression that uses one or more of the above - for example
  "Window.IsVisible(Videos) + !ListItem.IsResumable"

I measured the effect while the Videos window of the default skin was open
(but idle) on a Raspberry Pi, and this reduced the CPU usage by 3.2% from
45.1% to 41.9%:

          Before          After
          Mean   StdDev   Mean   StdDev  Confidence  Change
IdleCPU%  45.1   1.2      41.9   0.5     100.0%      +7.7%

[info] ensure we clear the window's conditional includes and visible condition on unload

[info] use std::find_if() to find previous infobool

[controls] GetVisibleCondition is only ever used to check if a visible condition exists, so replace with HasVisibleCondition

[info] Made CGUIInfoManager::Register() return a shared_ptr to the InfoBool object
rather than an index into its cache of such objects. All places where
those index integers were stored are changed to store shared_ptrs, and
CGUIInfoManager::GetBoolValue() now accepts them instead.

This permits the lifetime of a registered bool to persist across calls of
CGUIInfoManager::Clear(), if necessary.

[info] splits InfoSingle/Expression out from InfoBool.h

[info] ToLower() an infobool expression prior to searching to save unnecessary caseless comparisons

[info] CStdString -> std::string

CArchive: fix log report introduced in 643133d433bdeb9089c73887391d81dde85c763c

Revert "[nfs] - better fix for fe97d68f87d6985b2bf57d8e942b36b5e7373066 - possibly fixes #14727 - thx jm for the idea :)"

This reverts commit dd4dd2e91bc083a23fe7b46ae4c291c6672d32e0.

This commit created issues comparing paths when the export and the path was the same and did not contain a trailing slash, in addition, it removed the correct reversal of ordering of m_exportList generated by GetExportList.

Reference: http://trac.xbmc.org/ticket/14727#comment:12

Merge pull request #3750 from Karlson2k/update_utf8_check

Update UTF-8 string check

Merge pull request #3768 from taxigps/subtitle_addon

[subtitles] check and get idx/sub pair when download.

[lang] update of skin.confluence language files

[lang] update of core language files

[lang] update of internal addon language files

Merge pull request #3764 from FernetMenta/mpegts

ffmpeg: backport mpegts fix for dts jumping backwards (trac 14622)

VideoInfoScanner::EnumerateEpisodeItem: don't use ToLower with UTF-8 strings, fixes #14728
Not needed as RegExps are done in caseless mode

[coreSubs] proper fix for cf85bdb , select services list if no subtitles are found

[subtitles] check and get idx/sub pair when download.

Merge pull request #3699 from arnova/streamdetail_stack_time_fix

fixed: We can't grab the duration from the streamdetails for stacks

[nfs] - better fix for fe97d68f87d6985b2bf57d8e942b36b5e7373066 - possibly fixes #14727 - thx jm for the idea :)

[nfs] - get rid of some CStdStrings in favour of std::string

Merge pull request #3754 from popcornmix/fix_viewport_cache

[GLES] Fix for SBS rendering and cached viewport

Add missing includes of "system.h", fixes #14732

[ios/atv2] - readd gitrev-posix which was wrongly removed in 05274bfcbcbbc96a5ee5a61bf4018b30719727f4 - its still needed in ios/atv2 packaging if we want to retain dsym files (on jenkins...)

Merge pull request #3767 from Voyager1/fixmovieset

[fix] Could not edit/assign movie to movie set after #3642

[fix] Could not edit/assign movie to movie set after #3642
due to https://github.com/Montellese/xbmc/commit/e14fae38dec48cd1cd36f875f1ebde5396f43d57

Merge pull request #3733 from xhaggi/fix-epg-progress-indicator

[epg] fix: hide progress indicator if go forward in time

Merge pull request #3684 from Jalle19/pvr-small-fixes

Cleanup some of the PVR code

Merge pull request #3763 from xhaggi/fileitem-epginfotag-icon

[pvr] CFileItem: fallback to channel icon if epg icon is empty

Merge pull request #3702 from elmerfud/master

fix for #14713: RSSDirectory Reading of Boxee Genre Incorrect

ffmpeg: add patch for backport: mpegts-do-not-set-pts-for-missing-dts

mpegts: do not set pts for missing dts in video streams

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>

Merge pull request #3669 from jmarshallnz/right_means_right

Control positioning: Make <right>, <bottom> measure from right and bottom

[pvr] CFileItem: fallback to channel icon if epg icon is empty

Merge pull request #3755 from fritsch/revert-6d637d

Revert "Ensure that the decoder is init once only."

fixup! renderer: add underlying support for 16bit textures on gles

renderer: factor out logic for upgrading GL format to 16bit

fixup! renderer: calculate source rect on upload GLES:EGL/CVRef/MediaCodec

[confluence] update positioning so that <right> is measured from the right

drop <centerx> and <centery> and use <centerleft|right|top|bottom> where right and bottom measure from the right and bottom

<right> and <bottom> should measure from the right and bottom respectively

fixes incorrect position computation in the case where right and center are given

Merge pull request #3757 from Karlson2k/CArchive_upd_01

CArchive update

Merge pull request #3759 from Karlson2k/CArchive_incl_clean_01

CArchive include cleanup