Merge pull request #351 from AvaloniaCommunity/fixWarnings

Fix compilation warnings

Material.Ripple/RippleEffect.cs

@@ -56,7 +56,7 @@ protected override void OnSizeChanged(SizeChangedEventArgs e) {
             }
         }
 
-        private void PointerPressedHandler(object sender, PointerPressedEventArgs e) {
+        private void PointerPressedHandler(object? sender, PointerPressedEventArgs e) {
             var (x, y) = e.GetPosition(this);
             if (_container is null || x < 0 || x > Bounds.Width || y < 0 || y > Bounds.Height) {
                 return;
@@ -83,17 +83,17 @@ private void PointerPressedHandler(object sender, PointerPressedEventArgs e) {
             }
         }
 
-        private void LostFocusHandler(object sender, RoutedEventArgs e) {
+        private void LostFocusHandler(object? sender, RoutedEventArgs e) {
             _isCancelled = true;
             RemoveLastRipple();
         }
 
-        private void PointerReleasedHandler(object sender, PointerReleasedEventArgs e) {
+        private void PointerReleasedHandler(object? sender, PointerReleasedEventArgs e) {
             _isCancelled = true;
             RemoveLastRipple();
         }
 
-        private void PointerCaptureLostHandler(object sender, PointerCaptureLostEventArgs e) {
+        private void PointerCaptureLostHandler(object? sender, PointerCaptureLostEventArgs e) {
             _isCancelled = true;
             RemoveLastRipple();
         }

Material.Styles/Commands/SnackbarCommand.cs

@@ -16,9 +16,9 @@ public SnackbarCommand(SnackbarHost host, SnackbarModel model) {
             _model = model;
         }
 
-        public bool CanExecute(object parameter) => true;
+        public bool CanExecute(object? parameter) => true;
 
-        public void Execute(object parameter) {
+        public void Execute(object? parameter) {
             try {
                 _model.Button?.Action?.Invoke();
                 _host.SnackbarModels.Remove(_model);

Material.Styles/Controls/Clock.axaml.cs

@@ -115,8 +115,8 @@ protected override void OnApplyTemplate(TemplateAppliedEventArgs e) {
         UpdateSelectedTime();
     }
 
-    private void CircleClockPickerOnPointerReleased(object sender, PointerReleasedEventArgs e) {
-        var circleClockPicker = (CircleClockPicker)sender;
+    private void CircleClockPickerOnPointerReleased(object? sender, PointerReleasedEventArgs e) {
+        var circleClockPicker = (CircleClockPicker)sender!;
         if (circleClockPicker.Value is null) {
             return;
         }
@@ -136,8 +136,8 @@ private void CircleClockPickerOnPointerReleased(object sender, PointerReleasedEv
             ?.Focus();
     }
 
-    private void InputBoxOnGotFocusHandler(object sender, GotFocusEventArgs e) {
-        _selectedInputBox = (NumericUpDown)sender;
+    private void InputBoxOnGotFocusHandler(object? sender, GotFocusEventArgs e) {
+        _selectedInputBox = (NumericUpDown)sender!;
         OnSelectedInputBoxChange();
     }
 

Material.Styles/Controls/MaterialInternalIcon.axaml.cs

@@ -59,7 +59,7 @@ private void UpdateData() {
                 Data = Geometry.Parse(data);
 
             else
-                Data = null;
+                throw new InvalidOperationException($"There is no path data for icon kind {Kind}");
         }
     }
 }

Material.Styles/Controls/NavigationDrawer.axaml.cs

@@ -320,7 +320,7 @@ private void UpdateContentMargin()
             ContentMargin = new Thickness(left, 0, right, 0);
         }
 
-        private void PART_Scrim_Pressed(object sender, RoutedEventArgs e)
+        private void PART_Scrim_Pressed(object? sender, RoutedEventArgs e)
         {
             LeftDrawerOpened = false;
             RightDrawerOpened = false;

Material.Styles/Controls/Scroller.axaml.cs

@@ -244,7 +244,7 @@ private void CalculatedPropertiesChanged() {
             }
         }
 
-        private void OnLayoutUpdated(object sender, EventArgs e) => RaiseScrollChanged();
+        private void OnLayoutUpdated(object? sender, EventArgs e) => RaiseScrollChanged();
 
         private void RaiseScrollChanged() {
             var extentDelta = new Vector(Extent.Width - _oldExtent.Width, Extent.Height - _oldExtent.Height);

Material.Styles/Controls/SideSheet.axaml.cs

@@ -172,7 +172,7 @@ protected override void OnDetachedFromVisualTree(VisualTreeAttachmentEventArgs e
             base.OnDetachedFromVisualTree(e);
         }
 
-        private void PART_Scrim_Pressed(object sender, RoutedEventArgs e)
+        private void PART_Scrim_Pressed(object? sender, RoutedEventArgs e)
         {
             SideSheetOpened = false;
         }

Material.Styles/Controls/SnackbarHost.axaml.cs

@@ -132,7 +132,7 @@ public static void Post(SnackbarModel model, string? targetHost,
 
             // If duration is TimeSpan.Zero, dont expire it.
             if (model.Duration != TimeSpan.Zero) {
-                void OnExpired(object sender, ElapsedEventArgs args) {
+                void OnExpired(object? sender, ElapsedEventArgs args) {
                     if (sender is not Timer timer)
                         return;
 

Material.Styles/Themes/MaterialTheme.cs

@@ -65,7 +65,7 @@ public void Dispose() {
             _themeUpdateDisposable?.Dispose();
         }
 
-        private void OnOwnerChanged(object sender, EventArgs e) {
+        private void OnOwnerChanged(object? sender, EventArgs e) {
             RegisterActualThemeObservable();
         }
 
@@ -119,7 +119,7 @@ private void RegisterActualThemeObservable() {
             SetupActualTheme();
         }
 
-        private void HostOnActualThemeVariantChanged(object sender, EventArgs e) {
+        private void HostOnActualThemeVariantChanged(object? sender, EventArgs e) {
             SetupActualTheme();
         }