These are actually fairly convenient since they construct format objects a single time and then call the vectorized implementation with the format object. Still, it would be nice to get rid of all vectorized methods.
StefanKarpinski
All 4 comments
Maybe we could turn these into broadcast(::typeof(DateTime), ...) methods? That way the simple case would still be efficient and only build the DateFormat object once?
nalimilan
on 9 Aug 2017
Woah, those are pretty neat inline code snippets in the OP.
I actually don't think it should be an issue to remove these anymore, with the introduction of the dateformat"yyyy-mm-dd" DateFormat string macro, which ensures the DateFormat object is created only once, a la Regex.
quinnj
on 9 Aug 2017
We can remove these methods, and add the broadcast specializations later if they turn out being useful.
nalimilan
on 9 Aug 2017
Addressed by #23207. Best!
Sacha0
on 12 Aug 2017
Related issues
arshpreetsingh
路
3Comments
omus
路
3Comments
iamed2
路
3Comments
ararslan
路
3Comments
omus
路
3Comments
Most helpful comment
We can remove these methods, and add the
broadcastspecializations later if they turn out being useful.