This is the mail archive of the sid@sources.redhat.com mailing list for the SID project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

re: RFA: gloss component extensions


   
   Before you commit, I have some questions:

actually, i already have as frank approved it.
   
      2000-12-30  matthew green  <mrg@redhat.com>
   
      	* gloss.cxx (do_sys_time): New function.
      	(do_sys_times): Likewise.
      	(do_sys_gettimeofday): Likewise.
      	(syscall_trap): Call do_sys_time().  Call new do_sys_times() and
      	do_sys_gettimeofday() for times() and .
   
   `And ..' .. what?

` .. and gettimeofday, respectively.' is what the ChangeLog entry now says.
   
      + #else
      +   set_int_result(-1);
      + #endif
   
   In the case of missing gettimeofday() and times() functions, the newlib stub
   functions should set ENOSYS.  Do they use the -1 to detect this?

ah, i should probably also add a call:

	set_error_result(newlib::eNoSys);

then?
   
   Other than that, the only thing missing is your test cases to prove that
   these work.

actually, the entire gloss32 component has no testsuite at all.  :-(
(parts of the hardware specific ones do, but that is it.)

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]