Fix ip schema serialization logic

[sydney-runkle]

Fix #8343

[cloudflare-workers-and-pages]

Deploying pydantic-docs with    Cloudflare Pages

Latest commit:	e56b3f5
Status:	✅  Deploy successful!
Preview URL:	https://f2c2ebc0.pydantic-docs.pages.dev
Branch Preview URL:	https://fix-ip-ser.pydantic-docs.pages.dev

View logs

[codspeed-hq]

CodSpeed Performance Report

Merging #10112 will not alter performance

_{Comparing fix-ip-ser (e56b3f5) with main (cb3ac5a)}

Summary

✅ 17 untouched benchmarks

[github-actions]

Coverage report

Click to see where and how coverage changed

File	Statements	Missing	Coverage	Coverage (new stmts)	Lines missing
pydantic/_internal
_generate_schema.py
Project Total

This report was generated by python-coverage-comment-action

[Viicos]

LGTM. Perhaps ser_ip could be defined as a top level function to avoid being redefined every time. But not sure this has a huge impact on performance.

[sydney-runkle]

Yeah, I think that's a valid point. We seem to use this pattern a lot with custom schemas re ser functions within a function. Will leave as is for now, but I'd certainly be interested in some refactoring of _generate_schema.py that makes general changes like that.