[Gdcm-hackers] [gdcm] #141: sockstream::sync() is bogus
Cross-platform DICOM implementation
Brought to you by:
malat
|
From: gdcm - T. i. <no...@so...> - 2011-02-28 11:20:13
|
#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
|