Angus Leeming <[EMAIL PROTECTED]> writes:

| Lars Gullik Bj�nnes wrote:
| 
| > This patch fixes some GCC 3.3 warnings. If this is the best way to
| > do it can be argued.
| > 
| > Comments?
| 
| -        theCatcode['&']  = catAlign;
| +        theCatcode[static_cast<unsigned int>('\\')] = catEscape;
| 
| Why not:
| 
|         setCatcode('&', catAlign);

I have no idea, not my code. I only wanted the warnings gone.
 
| void setCatcode(char c, CATCODETYPE val) {
|         theCatcode[static_cast<unsigned char>(c)] = val;
| }
| 
| Saves polluting the source with loads of static_casts.

Sure, I'd be happy with that solution as well.

-- 
        Lgb

Reply via email to