[Tigervnc-commits] SF.net SVN: tigervnc:[4308] trunk/common/rfb/Hostname.h
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
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
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
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