On Mon, 2 Jul 2012, Tim Jenness wrote:
> 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).
Which I tend to leave alone, simply as that makes merging less hassle.
> 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.
I've pushed some updates to the code synthesis headers and updated the
generated code. Have a go with those (the setg fix was included). I don't
think an update to Xerces is needed, but try one if you like.
> 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).
Hmm, been a while but I would try defining CXX and doing all the build
support tools from clean sources. If that doesn't work you may need to
hack in the idea that clang++ is a sort of GCC compiler, if the options
are the same.
Cheers,
Peter.
|