[Tigervnc-commits] SF.net SVN: tigervnc:[4308] trunk/common/rfb/Hostname.h

2011-03-02 Thread ossman_
Revision: 4308
  http://tigervnc.svn.sourceforge.net/tigervnc/?rev=4308view=rev
Author:   ossman_
Date: 2011-03-02 12:44:46 + (Wed, 02 Mar 2011)

Log Message:
---
Catch when no host was specified a bit more gracefully.

Modified Paths:
--
trunk/common/rfb/Hostname.h

Modified: trunk/common/rfb/Hostname.h
===
--- trunk/common/rfb/Hostname.h 2011-03-02 12:44:12 UTC (rev 4307)
+++ trunk/common/rfb/Hostname.h 2011-03-02 12:44:46 UTC (rev 4308)
@@ -28,6 +28,8 @@
   static void getHostAndPort(const char* hi, char** host, int* port, int 
basePort=5900) {
 CharArray portBuf;
 CharArray hostBuf;
+if (hi == NULL)
+  throw rdr::Exception(NULL host specified);
 if (hi[0] == '[') {
   if (!strSplit(hi[1], ']', hostBuf.buf, portBuf.buf))
 throw rdr::Exception(unmatched [ in host);


This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.

--
Free Software Download: Index, Search  Analyze Logs and other IT data in 
Real-Time with Splunk. Collect, index and harness all the fast moving IT data 
generated by your applications, servers and devices whether physical, virtual
or in the cloud. Deliver compliance at lower cost and gain new business 
insights. http://p.sf.net/sfu/splunk-dev2dev 
___
Tigervnc-commits mailing list
Tigervnc-commits@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/tigervnc-commits


[Tigervnc-commits] SF.net SVN: tigervnc:[4309] trunk/common/rfb/ColourMap.h

2011-03-02 Thread ossman_
Revision: 4309
  http://tigervnc.svn.sourceforge.net/tigervnc/?rev=4309view=rev
Author:   ossman_
Date: 2011-03-02 12:45:57 + (Wed, 02 Mar 2011)

Log Message:
---
Provide a simple helper class for basic colour map usage.

Modified Paths:
--
trunk/common/rfb/ColourMap.h

Modified: trunk/common/rfb/ColourMap.h
===
--- trunk/common/rfb/ColourMap.h2011-03-02 12:44:46 UTC (rev 4308)
+++ trunk/common/rfb/ColourMap.h2011-03-02 12:45:57 UTC (rev 4309)
@@ -1,4 +1,5 @@
 /* Copyright (C) 2002-2005 RealVNC Ltd.  All Rights Reserved.
+ * Copyright 2011 Pierre Ossman oss...@cendio.se for Cendio AB
  * 
  * This is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -30,5 +31,20 @@
   public:
 virtual void lookup(int index, int* r, int* g, int* b)=0;
   };
+
+  class SimpleColourMap : public ColourMap {
+  public:
+SimpleColourMap(int size = 256) { table = new Colour[size]; };
+~SimpleColourMap() { delete [] table; };
+
+void lookup(int index, int* r, int* g, int* b)
+{ *r = table[index].r; *g = table[index].g; *b = table[index].b; };
+
+void set(int index, int r, int g, int b)
+{ table[index].r = r; table[index].g = g; table[index].b = b; };
+
+  protected:
+Colour *table;
+  };
 }
 #endif


This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.

--
Free Software Download: Index, Search  Analyze Logs and other IT data in 
Real-Time with Splunk. Collect, index and harness all the fast moving IT data 
generated by your applications, servers and devices whether physical, virtual
or in the cloud. Deliver compliance at lower cost and gain new business 
insights. http://p.sf.net/sfu/splunk-dev2dev 
___
Tigervnc-commits mailing list
Tigervnc-commits@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/tigervnc-commits


[Tigervnc-commits] SF.net SVN: tigervnc:[4311] trunk/configure.ac

2011-03-02 Thread ossman_
Revision: 4311
  http://tigervnc.svn.sourceforge.net/tigervnc/?rev=4311view=rev
Author:   ossman_
Date: 2011-03-02 15:41:22 + (Wed, 02 Mar 2011)

Log Message:
---
Make sure configure aborts if libjpeg isn't found.

Modified Paths:
--
trunk/configure.ac

Modified: trunk/configure.ac
===
--- trunk/configure.ac  2011-03-02 15:38:19 UTC (rev 4310)
+++ trunk/configure.ac  2011-03-02 15:41:22 UTC (rev 4311)
@@ -195,7 +195,7 @@
 AC_CONFIG_SUBDIRS([zlib])
 
 dnl Check for libjpeg library
-AC_SEARCH_LIBS([jpeg_destroy_compress], [jpeg])
+AC_SEARCH_LIBS([jpeg_destroy_compress], [jpeg], [], [AC_MSG_ERROR([libjpeg is 
required to build TigerVNC])])
 
 AC_CHECK_FUNCS([vsnprintf snprintf strcasecmp strncasecmp])
 


This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.

--
Free Software Download: Index, Search  Analyze Logs and other IT data in 
Real-Time with Splunk. Collect, index and harness all the fast moving IT data 
generated by your applications, servers and devices whether physical, virtual
or in the cloud. Deliver compliance at lower cost and gain new business 
insights. http://p.sf.net/sfu/splunk-dev2dev 
___
Tigervnc-commits mailing list
Tigervnc-commits@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/tigervnc-commits


Re: [Tigervnc-commits] SF.net SVN: tigervnc:[4310] trunk

2011-03-02 Thread DRC
As much as I support this move, I wish we had discussed it prior to
commit, as there is no way this will work as it is.  It needs to be more
rigorous to ensure that we're building against libjpeg-turbo, not
libjpeg (probably should check for the existence of the colorspace
extensions in the header.)  I also don't see any code that actually
performs the linking against libjpeg (?!)  There also needs to be a
mechanism added which will allow the C flags and LD flags to be
specified for building against/linking against libjpeg-turbo (similar to
the GNUTLS_CFLAGS and GNUTLS_LDFLAGS in configure.ac.

You also didn't remove the common/jpeg directory.




On 3/2/11 9:38 AM, ossm...@users.sourceforge.net wrote:
 Revision: 4310
   http://tigervnc.svn.sourceforge.net/tigervnc/?rev=4310view=rev
 Author:   ossman_
 Date: 2011-03-02 15:38:19 + (Wed, 02 Mar 2011)
 
 Log Message:
 ---
 Remove the included libjpeg-turbo from the tree and rely completely on one
 being provided by the system.
 
 Modified Paths:
 --
 trunk/common/CMakeLists.txt
 trunk/common/Makefile.am
 trunk/common/rfb/CMakeLists.txt
 trunk/common/rfb/Makefile.am
 trunk/configure.ac
 trunk/unix/README
 
 Removed Paths:
 -
 trunk/common/jpeg/
 
 Modified: trunk/common/CMakeLists.txt
 ===
 --- trunk/common/CMakeLists.txt   2011-03-02 12:45:57 UTC (rev 4309)
 +++ trunk/common/CMakeLists.txt   2011-03-02 15:38:19 UTC (rev 4310)
 @@ -1,6 +1,5 @@
  add_subdirectory(os)
  add_subdirectory(zlib)
 -add_subdirectory(jpeg)
  add_subdirectory(rdr)
  add_subdirectory(network)
  add_subdirectory(Xregion)
 
 Modified: trunk/common/Makefile.am
 ===
 --- trunk/common/Makefile.am  2011-03-02 12:45:57 UTC (rev 4309)
 +++ trunk/common/Makefile.am  2011-03-02 15:38:19 UTC (rev 4310)
 @@ -3,9 +3,6 @@
  if INCLUDED_ZLIB
  SUBDIRS += zlib
  endif
 -if INCLUDED_JPEG
 -SUBDIRS += jpeg
 -endif
  
  SUBDIRS += rdr network Xregion rfb
  
 
 Modified: trunk/common/rfb/CMakeLists.txt
 ===
 --- trunk/common/rfb/CMakeLists.txt   2011-03-02 12:45:57 UTC (rev 4309)
 +++ trunk/common/rfb/CMakeLists.txt   2011-03-02 15:38:19 UTC (rev 4310)
 @@ -1,5 +1,4 @@
 -include_directories(${CMAKE_SOURCE_DIR}/common ${CMAKE_SOURCE_DIR}/win
 -  ${CMAKE_SOURCE_DIR}/common/jpeg ${CMAKE_SOURCE_DIR}/common/jpeg/win)
 +include_directories(${CMAKE_SOURCE_DIR}/common ${CMAKE_SOURCE_DIR}/win)
  
  set(RFB_SOURCES
Blacklist.cxx
 
 Modified: trunk/common/rfb/Makefile.am
 ===
 --- trunk/common/rfb/Makefile.am  2011-03-02 12:45:57 UTC (rev 4309)
 +++ trunk/common/rfb/Makefile.am  2011-03-02 15:38:19 UTC (rev 4310)
 @@ -62,11 +62,6 @@
  librfb_la_LIBADD += $(PAM_LIBS)
  endif
  
 -if INCLUDED_JPEG
 -librfb_la_CPPFLAGS += -I$(top_srcdir)/common/jpeg 
 -I$(top_builddir)/common/jpeg
 -librfb_la_LIBADD += $(top_builddir)/common/jpeg/libjpeg.la
 -endif
 -
  if BUILD_WIN
  librfb_la_SOURCES += WinPasswdValidator.h WinPasswdValidator.cxx
  endif
 
 Modified: trunk/configure.ac
 ===
 --- trunk/configure.ac2011-03-02 12:45:57 UTC (rev 4309)
 +++ trunk/configure.ac2011-03-02 15:38:19 UTC (rev 4310)
 @@ -10,7 +10,7 @@
  AC_CONFIG_AUX_DIR([.])
  AM_INIT_AUTOMAKE([-Wall foreign dist-bzip2])
  
 -AC_CONFIG_SUBDIRS([common/jpeg common/zlib])
 +AC_CONFIG_SUBDIRS([common/zlib])
  
  AC_CONFIG_HEADERS([config.h])
  
 @@ -195,17 +195,8 @@
  AC_CONFIG_SUBDIRS([zlib])
  
  dnl Check for libjpeg library
 -INCLUDED_JPEG=yes
 -AC_ARG_WITH([system-jpeg],
 - AS_HELP_STRING([--with-system-jpeg], 
 -[use libjpeg which is distributed with the O/S]),
 - [AC_SEARCH_LIBS([jpeg_destroy_compress], [jpeg],
 - [INCLUDED_JPEG=no], [])],
 - [])
 +AC_SEARCH_LIBS([jpeg_destroy_compress], [jpeg])
  
 -AM_CONDITIONAL([INCLUDED_JPEG], [ test x$INCLUDED_JPEG = xyes ])
 -AC_CONFIG_SUBDIRS([jpeg])
 -
  AC_CHECK_FUNCS([vsnprintf snprintf strcasecmp strncasecmp])
  
  # Network functions
 
 Modified: trunk/unix/README
 ===
 --- trunk/unix/README 2011-03-02 12:45:57 UTC (rev 4309)
 +++ trunk/unix/README 2011-03-02 15:38:19 UTC (rev 4310)
 @@ -61,11 +61,6 @@
  This should first build some libraries - rdr, network, Xregion, rfb and tx -
  followed by the vncviewer, vncconfig and vncpasswd applications.
  
 -The default is to build an optimized version of libjpeg included with the
 -TigerVNC source and link these applications against it.  To instead link
 -against the version of libjpeg provided by the operating system, pass an
 -argument of --with-system-jpeg to configure.
 -
  The default is to link against the version of zlib