Respond to CR.

SignalMessaging/Views/ImageEditor/ImageEditorModel.swift

@@ -340,8 +340,13 @@ private class ImageEditorOperation: NSObject {
 
 @objc
 public protocol ImageEditorModelDelegate: class {
+    // Used for large changes to the model, when the entire
+    // model should be reloaded.
     func imageEditorModelDidChange(before: ImageEditorContents,
                                    after: ImageEditorContents)
+
+    // Used for small narrow changes to the model, usually
+    // to a single item.
     func imageEditorModelDidChange(changedItemIds: [String])
 }
 

SignalMessaging/Views/ImageEditor/ImageEditorView.swift

@@ -196,7 +196,7 @@ public class ImageEditorView: UIView, ImageEditorModelDelegate {
         case .brush:
             handleBrushGesture(gestureRecognizer)
         case .crop:
-                handleCropGesture(gestureRecognizer)
+            handleCropGesture(gestureRecognizer)
         }
     }
 

SignalMessaging/utils/OWSMath.h

@@ -4,7 +4,6 @@
 
 NS_ASSUME_NONNULL_BEGIN
 
-// TODO: We'll eventually want to promote these into an OWSMath.h header.
 CG_INLINE CGFloat CGFloatClamp(CGFloat value, CGFloat minValue, CGFloat maxValue)
 {
     return MAX(minValue, MIN(maxValue, value));