search

drewnoakes / metadata-extractor-dotnet

Extracts Exif, IPTC, XMP, ICC and other metadata from image, video and audio files
Other
922 stars 164 forks source link

Use file scoped namespaces #353

Closed iamcarbon closed 8 months ago

iamcarbon commented 8 months ago

@drewnoakes Ready for feedback / review. This PR contains no functional changes.

File scoped namespaces improve the codebase readability, particularly on smaller screens (like laptops).

iamcarbon commented 8 months ago

Here's a direct link to review, ignoring whitespace changes:

https://github.com/drewnoakes/metadata-extractor-dotnet/pull/353/files?diff=unified&w=1

drewnoakes commented 8 months ago

Thanks but I'm not inclined to change this. Such broad changes can interfere with trawling through history to understand why code exists as is.. While I like the language feature, I'd prefer to only use it for new code.