From: Yasir A. <li...@en...> - 2005-11-04 20:16:01
|
Hi Jeff, >>Have any of the MDB Tools developers had a chance to look at my patch >>(below)? I sent this email in a few weeks ago. >> >> > >Thank you for the patch, I now have it committed. I fixed several other things in >neighboring functions at the same time. > > > That's excellent - thanks. >>>Just in case this isn't clear, the change is in data.c, function >>>mdb_xfer_bound_bool(), with the new code inserted at line 174. >>> >>> > >By the way, as with most open-source projects, we prefer patches in 'unified' >format. In case you don't know, you do this by adding the -u option to the >diff command. For instance 'cvs diff -u data.c'. It is easier to read >(in my opinion at least) and it gives some lines of context in case line numbers >in different people's versions of a file don't match up exactly. > > I didn't know this - I'm glad you told me (this is the first patch I've ever sent to an open source project!) Thanks again for all the work you and the other developers put into this product - I really appreciate it. Yasir |