List:MySQL++« Previous MessageNext Message »
From:Warren Young Date:September 19 2005 7:51pm
Subject:Re: [patch] row::operator[] - template version
View as plain text  
Warren Young wrote:
> 
> I have tested it on both g++ on Linux and VC++ 7.1 on Windows, and it 
> seems to work.

I overlooked the fact that this will break the ABI, while the other 
patch will not.  Rather than endorse the other one, which I really don't 
like, I'm now in favor of putting this on the v3.0 wishlist, where we 
can break the ABI at will.

A solution we shouldn't overlook is simply changing the SizeType 
definition at the const_subscript_iterator level to 'int'.  Negative 
subscripts don't make sense, but the range checking in vector::at() will 
take care of this.  Then we don't need ugly overloads, and can avoid the 
risks pps brought up.
Thread
[patch] row::operator[] - template versionChris Frey18 Sep
  • Re: [patch] row::operator[] - template versionWarren Young19 Sep
    • Re: [patch] row::operator[] - template versionWarren Young19 Sep
      • Re: [patch] row::operator[] - template versionChris Frey19 Sep
        • Re: [patch] row::operator[] - template versionWarren Young19 Sep
          • Re: [patch] row::operator[] - template versionChris Frey19 Sep
Re: [patch] row::operator[] - template versionChris Frey18 Sep
  • Re: [patch] row::operator[] - template versionUnknown Sender18 Sep
    • Re: [patch] row::operator[] - template versionChris Frey18 Sep
      • Re: [patch] row::operator[] - template versionUnknown Sender18 Sep
        • Re: [patch] row::operator[] - template versionWarren Young19 Sep
          • Re: [patch] row::operator[] - template versionUnknown Sender19 Sep
            • Re: [patch] row::operator[] - template versionWarren Young19 Sep
      • Re: [OBORONA-SPAM] Re: [patch] row::operator[] - template versionUnknown Sender19 Sep
        • Re: [OBORONA-SPAM] Re: [patch] row::operator[] - template versionChris Frey19 Sep
          • Re: [patch] row::operator[] - template versionUnknown Sender19 Sep
            • Re: [patch] row::operator[] - template versionChris Frey19 Sep
            • Re: [patch] row::operator[] - template versionWarren Young19 Sep
        • Re: [patch] row::operator[] - template versionWarren Young19 Sep
    • Re: [patch] row::operator[] - template versionChris Frey18 Sep
  • Re: [patch] row::operator[] - template versionWarren Young19 Sep