Update iPlug2 submodule with goniometer control

[olilarkin]

Adds IVGoniometerControl for stereo field visualization - a vectorscope
that displays stereo audio as an X-Y Lissajous pattern with persistence.

[claude]

Pull Request Review

Summary

This PR updates the iPlug2 submodule to include the new IVGoniometerControl for stereo field visualization. The change is minimal (submodule pointer update from 608f8d8 to c2073ef).

Code Quality & Best Practices ✅

• Clean commit message: The commit message clearly describes what's being added
• Minimal change: Submodule updates are appropriately atomic
• Co-authored attribution: Properly attributes the commit to Claude

Potential Issues & Considerations

1. Submodule Update Verification ⚠️

• The PR description mentions adding IVGoniometerControl, but without access to the actual submodule commits, I cannot verify:
  • What other changes (if any) are included in this submodule update
  • Whether this is a targeted update or includes unrelated changes
  • The quality of the goniometer implementation itself

Recommendation: Before merging, verify that the submodule update range (608f8d8..c2073ef) contains only the intended goniometer changes, or document any additional changes included.

2. Testing Considerations 🧪

• No test coverage information is visible in this repository for the submodule changes
• For audio visualization controls, consider testing:
  • Edge cases (mono input, null input, extreme phase relationships)
  • Performance with different buffer sizes
  • Memory usage with persistence feature
  • Thread safety if used in real-time audio context

3. Documentation 📝

• Ensure the iPlug2 submodule includes documentation for the new IVGoniometerControl
• Consider adding an example usage in one of the template projects

Security Concerns 🔒

• Submodule updates generally have low security risk
• Audio visualization should not introduce security vulnerabilities if implemented correctly
• Ensure the goniometer control properly validates input buffer sizes to prevent buffer overruns

Performance Considerations ⚡

• Goniometers with persistence can be CPU-intensive depending on implementation
• Verify that the control uses efficient rendering (e.g., vertex batching, proper invalidation)
• Consider whether the persistence feature can be optionally disabled for lower-end systems

Repository-Specific Notes

Per CLAUDE.md, this repository maintains iPlug2 as a submodule for the "out of source" plugin development workflow. This update pattern is consistent with the repository's design.

Overall Assessment

Status: ✅ Approve with minor recommendations

This is a straightforward submodule update that appears to add useful functionality. The main recommendation is to verify the exact scope of changes in the submodule update range before merging.

---

Review generated by Claude Code