[Gdcm-hackers] [gdcm] #141: sockstream::sync() is bogus

[gdcm - T. i. <no...@so...>]

#141: sockstream::sync() is bogus
---------------------+------------------------------------------------------
 Reporter:  malat    |       Owner:  malat                                    
     Type:  defect   |      Status:  new                                      
 Priority:  blocker  |   Milestone:  [MM-10] - DICOM protocol for PACS support
Component:  network  |     Version:  2.0                                      
 Keywords:           |  
---------------------+------------------------------------------------------
 Currently suckbuf implementation is bogus. We should not try to write more
 data during a sync() without checking if write is possible


 int sockbuf::sync ()
 // we never return -1 because we throw sockerr
 // exception in the event of an error.
 {
     if (pptr () && pbase () < pptr () && pptr () <= epptr ())
     {
         // we have some data to flush
         try {
             write (pbase (), pptr () - pbase ());
         }

-- 
Ticket URL: <http://sourceforge.net/apps/trac/gdcm/ticket/141>
gdcm <http://sourceforge.net/projects/gdcm/>
SF-project gdcm