Cvar parser...

[GoogleCodeExporter]

Im requesting that a fix be found for it...or a request for it to be
removed, lets keep this client features working please :|

Original issue reported on code.google.com by I.Am.Gew...@gmail.com on 8 Dec 2008 at 6:30

[GoogleCodeExporter]

What's wrong with it?

Original comment by amanieu on 8 Dec 2008 at 7:22

[GoogleCodeExporter]

theres an issue? works great for me more info please

Original comment by SlackerL...@gmail.com on 8 Dec 2008 at 9:51

[GoogleCodeExporter]

tried the cvar parser with my player status binds worked great it got your cvar 
info
fine and it substituted the cvar values in text.

Original comment by SlackerL...@gmail.com on 9 Dec 2008 at 2:31

• Changed state: New

[GoogleCodeExporter]

Set a bind with \$$cvar\, exit trem, open it, exit, open it, the bind is gone

Original comment by I.Am.Gew...@gmail.com on 9 Dec 2008 at 4:20

[GoogleCodeExporter]

i think i see something but its not with the \$$cvar\ bind whenever i clsoe its
losing my new cvar i set with /set test 5 :( ill look into it mustve been an 
ioq3
merge or something

Original comment by SlackerL...@gmail.com on 9 Dec 2008 at 5:32

[GoogleCodeExporter]

Use /seta to keep the cvar between restarts (saves it to autogen)

Original comment by amanieu on 9 Dec 2008 at 5:34

[GoogleCodeExporter]

Gewgle is right, it's been that way since the introduction of this patch. The 
only
way to make it stick is to put it in an autoexec; it's expanded if it's not put 
in a
.cfg file of some sort (other than the autogen, which it's already expanded to 
anyways).

Original comment by benburh...@gmail.com on 9 Dec 2008 at 5:56

[GoogleCodeExporter]

since i think thats stupid i looked into it and it only saved archive ones (not 
very
well too cause i tried seta also) so heres a patch that copys some of the 
archive
saving part and makes it save user generated cvars using set. don't see why it
shouldn't work but try it out works here with a 5 min test ill be testing it 
more later

Original comment by SlackerL...@gmail.com on 9 Dec 2008 at 6:05

• Changed state: PatchAvailable

Attachments:

• cvarfix.patch

[GoogleCodeExporter]

While you guys are messing with this, what about removing the backslashes?  
It's not like anyone would ever use 
$$ for anything else, anyway...

Original comment by doomagent13@yahoo.com on 11 Dec 2008 at 8:45

[GoogleCodeExporter]

possibly but an issue might come up when people use the dollar sign and it kinda
groups the whole thing together atm

Original comment by SlackerL...@gmail.com on 12 Dec 2008 at 1:54

[GoogleCodeExporter]

Original comment by SlackerL...@gmail.com on 13 Dec 2008 at 3:10

• Changed state: Applied
• Added labels: Revision-160

[GoogleCodeExporter]

did that fix your issues google?

Original comment by SlackerL...@gmail.com on 13 Dec 2008 at 3:11

[GoogleCodeExporter]

Yes, it seems to have fixed the problem, but it also reset ALL of my cvars to 
their
defaults, as well as aliases, when I recompiled. No idea why, and it seems that 
they
stick again once I customize them, but I lost my HUD, my custom console, and a 
bunch
of important stuff like cl_guidserveruniq... be sure to reset this once you 
recompile
for obvious security reasons. Amanieu and I couldn't figure out why it did 
this, any
ideas? Ubuntu 8.10 64-bit, updated to r161 from r156.

Original comment by benburh...@gmail.com on 13 Dec 2008 at 8:50

[GoogleCodeExporter]

worth looking into. while coding it i lost my autogen but shouldn't make a 
difference
since all it does to load the autogen is exec autogen.cfg???

Original comment by SlackerL...@gmail.com on 13 Dec 2008 at 9:29

[GoogleCodeExporter]

I think I know what the problem is. When it's "execing autogen.cfg" do you see
anything like "Cbuf_InsertText: overflow"?

Original comment by amanieu on 13 Dec 2008 at 9:56

[GoogleCodeExporter]

yes it just screwed up and i got that so there's definitively an issue

Original comment by SlackerL...@gmail.com on 13 Dec 2008 at 12:25

[GoogleCodeExporter]

What's the size of your autogen.cfg?

Original comment by amanieu on 13 Dec 2008 at 1:02

[GoogleCodeExporter]

http://www.tremfusion.net/hg/tremfusion/rev/268041bc0263
This should also help.

Original comment by amanieu on 13 Dec 2008 at 1:08

[GoogleCodeExporter]

its late so ill look into doing something about it in the morning ive reverted 
the
code till its all sorted.

took a look at that revision heres a .patch of it. doesn't seem to save them so 
its
really the same as before. since autogen doesn't have any user created i could 
prob
setup another file that shouldn't break anything and should help with the issue
instead of dumping it all in autogen.cfg.

Original comment by SlackerL...@gmail.com on 13 Dec 2008 at 2:53

• Changed state: PatchAvailable

Attachments:

• cvarfix2.patch