Re: [patch #5637] Implement DATESUM, DATEDIFF expression function

[Daniel E WILLIAMS]

Thanks.  I think the arguments are backwards in the function call, if you are trying for SPSS compatibility.  SPSS uses the format DATEDIFF(last_date, first_date, unit).  I think you implemented DATEDIFF(first_date, last_date, unit).  I tried to find this in the code to fix myself, but am stumped.  Can you tell me where to look?  Thanks.

 

 

 

 

Dan Williams
Forecasting, Research and Analysis Office
Department of Human Services
State of Oregon, USA
503 947 5354

>>> Ben Pfaff <address@hidden> 12/13/2006 7:37:15 PM >>>

Update of patch #5637 (project pspp):

                  Status:                    None => Done                  
             Open/Closed:                    Open => Closed                

    _______________________________________________________

Follow-up Comment #2:

>I didn't closly inspect the results in the tests.

I'm pretty sure of the results as far as they go, because I generated them by
hand.  Could easily be I didn't test some important case, but that's the way
it goes.

Daniel, I checked this in.  Let me know if it doesn't work for you.

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/patch/?5637>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/