search

jsgoupil / quickbooks-sync

Sync Quickbooks Desktop
MIT License
89 stars 40 forks source link

Update DATETYPE.cs #8

Closed cyberjared closed 5 years ago

cyberjared commented 9 years ago

Changed incorrect operator || to ^

jsgoupil commented 9 years ago

Although I agree with your changes based on the comment. I copy pasted this from Microsoft Guidelines. https://msdn.microsoft.com/en-US/library/ms173147(v=vs.80).aspx

I'm ok to change it but it's used at multiple places. So it would be better to change it everywhere.

Thanks.

mscappini commented 9 years ago

I've never seen this approach. This is clever.

Yeah, it doesn't look like the Microsoft Guidelines are accurate in this case. Odd.

How are these generated? Can we just change a template?