deprecate readcsv in favor of readdlm

[Sacha0]

This pull request deprecates readcsv(io[, T::Type]; opts...) in favor of readdlm(io, ','[, T]; opts...), addressing part of #23414. Best!

[dpsanders]

Isn't readcsv a more obvious name than readdlm?

[JeffBezanson]

That's the problem. You shouldn't use this function, because a far better implementation exists in a package.

[dpsanders]

I meant that the name readdlm is hard to discover and hard to remember.

…

On 31 Aug 2017 3:39 p.m., "Jeff Bezanson" ***@***.***> wrote: That's the problem. You shouldn't use this function, because a far better implementation exists in a package. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#23530 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AALtThlpiqkxYyikqn27ATUFzW7pitHJks5sdxoYgaJpZM4PJOhL> .

[StefanKarpinski]

I meant that the name readdlm is hard to discover and hard to remember.

True. I'm all for deleting readdlm as well.

[ViralBShah]

So, just to load some simple files in Base, one has to go mucking with packages? If the name readdlm is hard to discover, it makes sense to rename it with something better. However, I don't understand this obsession with deleting everything from Base.

[Sacha0]

Thanks all! :)