Class name prefix? about htmlreader HOT 7 OPEN

Hello!

Which projects are you aware of that suffer from name collisions?

I kinda consider HTML the class prefix for HTMLReader. When I started this project I looked around to see if anyone else was using it. I didn't find anyone, so I proceeded to use it. Didn't seem too general at the time!

But I'm not surprised to hear that there's some clashing, and in principle I'm for eliminating pointless friction.

from htmlreader.

I would agree that HTML is probably too generic for use as a class prefix, and is likely to cause issues in some projects (e.g., if Apple were to introduce some HTML classes into Swift's standard library). Apple's Objective-C Conventions also state that class prefixes should be 3 letters:

"Your own classes should use three letter prefixes. These might relate to a combination of your company name and your app name, or even a specific component within your app."

from htmlreader.

I’m not sure they meant "and only three letter prefixes". The immediate context for the snippet you quote is:

Two-letter prefixes like these are reserved by Apple for use in framework classes.

Besides, I can’t think of a reasonable three letters to choose as a prefix ;)

Anyway, my stance is to leave it be until someone can point to a project that’s causing conflicts. Once that happens I’m open to working something out.

from htmlreader.

Totally reasonable. And thank you for this library. It's been completely rock solid and saved me a ton of time!

from htmlreader.

You’re quite welcome!

from htmlreader.

I do have a naming conflicts with pod 'TapjoySDK' with following message:

duplicate symbol _parseNumber in:
/Users/user/Library/Developer/Xcode/DerivedData/MyProject-dccczrsfcebodlhhgwfuljlwdtfq/Build/Products/Debug-iphoneos/HTMLReader/libHTMLReader.a(HTMLSelector.o)
Pods/TapjoySDK/TapjoySDK_iOS_v11.12.2/Libraries/Tapjoy.embeddedframework/Tapjoy.framework/Tapjoy(Tapjoy)
ld: 1 duplicate symbol for architecture arm64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

Please add a prefix :)

from htmlreader.

@lurui1029 I'ma move that to a new issue if that's alright! #85

from htmlreader.