search

byronwall / bUTL

Excel add-in with helpers for charting, formatting, and general pain points
http://byroni.us/bUTL
MIT License
16 stars 3 forks source link

Some more cleanup #50

Closed RaymondWise closed 8 years ago

byronwall commented 8 years ago

What changed in these? I see the capitalization difference on NewSeries -> newSeries. Since that's a native function call, I don't think that needs camelCase. I would also expect the VBE to fight this change since it tries to correct case on native calls.

Let me know if I am missing something (or if the frx files have changes that don't show up in the text files).

Thanks for continuing to suggest improvements here. I'm saying it too frequently, but I am sorry for the delay in getting through the PRs...

RaymondWise commented 8 years ago

Huh, yeah. All the changes were in the bUTLChartSeries class module, but I don't remember what. I'm going to withdraw and rebuild from source.

RaymondWise commented 8 years ago

Also, that change would be a stupid one to make. I think I was doing some search/replace stuff but I don't remember what happened. I've rebuilt now.