river: clean up some uneeded TODOs

river/Cursor.zig

@@ -224,7 +224,6 @@ pub fn init(self: *Self, seat: *Seat) !void {
     wlr_cursor.events.pinch_end.add(&self.pinch_end);
     seat.wlr_seat.events.request_set_cursor.add(&self.request_set_cursor);
 
-    // TODO(wlroots) handle the cancel event, blocked on wlroots 0.16.0
     wlr_cursor.events.touch_up.add(&self.touch_up);
     wlr_cursor.events.touch_down.add(&self.touch_down);
     wlr_cursor.events.touch_motion.add(&self.touch_motion);

river/InputConfig.zig

@@ -29,11 +29,6 @@ const util = @import("util.zig");
 
 const InputDevice = @import("InputDevice.zig");
 
-// TODO - keyboards
-//      - mapping to output / region
-//      - calibration matrices
-//      - scroll factor
-
 pub const EventState = enum {
     enabled,
     disabled,

river/Root.zig

@@ -740,8 +740,6 @@ fn processOutputConfig(
 }
 
 fn currentOutputConfig(self: *Self) !*wlr.OutputConfigurationV1 {
-    // TODO there no real reason this needs to allocate memory every time it is called.
-    // consider improving this wlroots api or reimplementing in zig-wlroots/river.
     const config = try wlr.OutputConfigurationV1.create();
     // this destroys all associated config heads as well
     errdefer config.destroy();

river/XwaylandView.zig

@@ -115,7 +115,6 @@ pub fn configure(self: Self) bool {
 }
 
 pub fn rootSurface(self: Self) *wlr.Surface {
-    // TODO This is probably not OK, understand when xwayland surfaces can be null.
     return self.xwayland_surface.surface.?;
 }