search

tmontaigu / dbase-rs

Rust library to read & write dBase files.
MIT License
29 stars 30 forks source link

implement Default to Date: 1900/1/1 #64

Closed gdennie closed 1 year ago

gdennie commented 1 year ago

default values are ergonomic

tmontaigu commented 1 year ago

I would say that default should be init to todays date

What do you think ?

gdennie commented 1 year ago

I don't think so. The value should be recognizably not set. Another suitable default value could be 0000/01/01, for instance. The purpose of default value is ergonomics. Additionally, today's date is rarely the value you want anyway.

tmontaigu commented 1 year ago

I'm ok for default to be 1900/1/1

I would prefer another PR for the other fix tho :smiley: