Line 26 of Contrib\Modern UI 2\Pages\Finish.nsh is
!ifndef MUI_FINISHPAGE_NOREBOORTSUPPORT
I think it's meant to read
!ifndef MUI_FINISHPAGE_NOREBOOTSUPPORT
otherwise the "mui.FinishPage.ReturnValue" is always created, even when it doesn't need to be. Which causes the following warning in the compilier
Variable "mui.FinishPage.ReturnValue" not referenced or never set, wasting memory!
Logged In: YES
user_id=584402
Originator: NO
Thanks, fixed.