On Mon, Jul 2, 2012 at 6:32 AM, Peter W. Draper <[log in to unmask]> wrote:
> On Sun, 10 Jun 2012, Peter W. Draper wrote:
>
>>> Easiest possible way - a popup asking for the filename and then
>>> displaying it. I suppose the issue is whether the toolbox should allow
>>> regions to be enabled and disabled and whether the toolbox should support
>>> multiple regions.
>>
>>
>> I'll make a start on something simple and we can see how that goes.
>
>
> Tried to do that. Added a new toolbox to the Image-Analysis menu. In this
> you just write STC-S descriptions and press Draw. You change the colour and
> thickness of the outline, save and restore descriptions to disk file.
> Multiple descriptions are separated by an empty line and you can clone the
> toolbox so draw from several windows (usual way). Let me know if anyone
> tries this out.
Thanks. I'll take a look.
>
> Also note I've incremented the GAIA version number, so I'd recommend a git
> clean in the applications/gaia directory this time around.
>
I've pushed a few cleanups for compiler warnings. Main remaining
warnings are from Tcl_SetResult and Skycat (which shouldn't be
installing its own outdated fitsio.h but there are also other problems
with consting in skycat).
I've also had another go at using clang and clang++ to build gaia
(version 3.1 of the clang compilers). clang++ really doesn't like the
automatically generated xsd code for xerces. I'm wondering if we need
to update xerces and regenerate the C++ code.
In file included from ./xsd/cxx/zc-istream.hxx:90:
./xsd/cxx/zc-istream.txx:35:7: error: use of undeclared identifier 'setg'
setg (b, b, e);
^
this->
./xsd/cxx/zc-istream.txx:17:7: note: in instantiation of member
function 'xsd::cxx::zc_streambuf<char>::init' requested here
init ();
^
./xsd/cxx/zc-istream.txx:64:11: note: in instantiation of member
function 'xsd::cxx::zc_streambuf<char>::zc_streambuf' requested here
: buf_ (str)
^
./xsd/cxx/zc-istream.txx:81:11: note: in instantiation of member
function 'xsd::cxx::zc_istream_base<char>::zc_istream_base' requested
here
: zc_istream_base<C> (str),
^
./xsd/cxx/tree/parsing/unsigned-long.hxx:69:23: note: in instantiation
of member function 'xsd::cxx::zc_istream<char>::zc_istream' requested
here
zc_istream<C> is (tmp);
^
./xsd/cxx/tree/parsing/unsigned-long.hxx:51:16: note: in instantiation
of member function 'xsd::cxx::tree::traits<unsigned long long, char,
0>::create' requested here
return create (xml::transcode<C> (a.getValue ()), 0, f, c);
^
./generic/VOTable1.1.C:5231:27: note: in instantiation of member
function 'xsd::cxx::tree::traits<unsigned long long, char, 0>::create'
requested here
this->width_.set (width_traits::create (i, f, this));
^
/usr/include/c++/4.2.1/streambuf:504:7: note: must qualify identifier
to find this declaration in dependent base class
setg(char_type* __gbeg, char_type* __gnext, char_type* __gend)
^
1 error generated.
Changing the code to use this->setg seems to fix the compiler issue.
My final problem is that libtool doesn't understand that clang++ is a
C++ compiler when it comes to linking. I'm wondering if that means I
need to rebuild my buildsupport libtool (although I can never work out
how it knows what types of compilers we have and updating our ancient
libtool is another todo item).
--
Tim Jenness
|