While(false) nov19 logs-oop344-20113

From CDOT Wiki
Revision as of 01:03, 20 November 2011 by Lpasquale1 (talk | contribs) (Created page with '<pre> Session Start: Sat Nov 19 14:56:15 2011 Session Ident: psupreme [14:56] <BriaN_90> hey do u have time? [14:57] <BriaN_90> i need someone to debug with me the CButton [14:57…')
(diff) ← Older revision | Latest revision (diff) | Newer revision → (diff)
Jump to: navigation, search
Session Start: Sat Nov 19 14:56:15 2011
Session Ident: psupreme
[14:56] <BriaN_90> hey do u have time?
[14:57] <BriaN_90> i need someone to debug with me the CButton
[14:57] <BriaN_90> something doesn't work properly
[15:01] <psupreme> ok
[15:01] <psupreme> did you merge?
[15:01] <BriaN_90> no i didn't merge yet
[15:01] <BriaN_90> i wanna finish the cbutton and merge
[15:01] <BriaN_90> i think i have a problem with the draw
[15:01] <BriaN_90> oh but wait
[15:01] <BriaN_90> i didn't commit
[15:01] <BriaN_90> the vxproj
[15:03] <BriaN_90> ok now it's ready
[15:04] <psupreme> okay, so whats wrong
[15:05] <BriaN_90> the first problem
[15:05] <BriaN_90> it doesn't draw properly in the border
[15:05] <BriaN_90> i followed the instructions on the webesite
[15:05] <BriaN_90> website
[15:06] <BriaN_90> and the other problem is the edit
[15:06] <BriaN_90> it should remove the brackets
[15:06] <BriaN_90> when i change
[15:06] <BriaN_90> button but it doesn't
[15:07] <psupreme> visible means you are selected?
[15:07] <psupreme> the button is selected*
[15:07] <BriaN_90> u r talking about the CFrame::visible()?
[15:07] <psupreme> ya
[15:08] <BriaN_90> visible should return true
[15:08] <BriaN_90> if the frame has the border
[15:08] <BriaN_90> false otherwise
[15:08] <BriaN_90> i think lol
[15:08] <psupreme> ok
[15:08] <psupreme> first bug fixed
[15:08] <psupreme> in ctor
[15:08] <psupreme> if(Bordered
[15:08] <psupreme> should be !Bordered
[15:08] <psupreme> and you should offset border accordingly
[15:09] <psupreme> ie. put the text in middle NOT @ 0,0
[15:10] <BriaN_90> why !Bordered?
[15:10] <psupreme> if its bordered it needs a bigger height
[15:10] <psupreme> so the border displays properly
[15:10] <psupreme> you just have it reversed
[15:11] <BriaN_90> oooh right
[15:12] <BriaN_90> but
[15:12] <BriaN_90> if u read
[15:12] <BriaN_90> For width: Set width to the length of Str + 2 (adding 2 for surrounding brackets) or if the Button is bordered set width to the length of Str + 4 (adding 2 for surrounding brackets and 2 for the borders). For height: Set the height to 1 or if the Button is bordered, set the height to 3.
[15:12] <BriaN_90> that's what i did
[15:12] <BriaN_90> uhm
[15:13] <psupreme> okay
[15:13] <psupreme> in draw
[15:13] <psupreme> if(!CFrame::visible())
[15:13] <psupreme> console.display((char *)_data,CFrame::absRow(),CFrame::absCol());
[15:13] <psupreme> else
[15:13] <psupreme> console.display((char *)_data,CFrame::absRow()+1,CFrame::absCol()+1);
[15:13] <psupreme> should be that
[15:13] <psupreme> but....
[15:13] <psupreme> the button is too small
[15:13] <psupreme> not wide enough to store the brackets
[15:14] <BriaN_90> uhm on the website it said that it has to be +2 O.o
[15:14] <psupreme> yeah, put +2 back
[15:14] <psupreme> sry
[15:15] <BriaN_90> ok
[15:15] <psupreme> but at the top
[15:15] <psupreme> gotta do
[15:15] <psupreme> width(strlen(Str)+4);
[15:15] <psupreme> in the else
[15:16] <BriaN_90> why?
[15:16] <psupreme> so the border is wide enough
[15:16] <BriaN_90> in this case both are the same, just the height is different
[15:16] <BriaN_90> so the first part of the if
[15:16] <BriaN_90> should be +2?
[15:18] <psupreme> whats the flag to check if its selected?
[15:18] <psupreme> because i think edit is broken
[15:18] <BriaN_90> there is no flag
[15:18] <BriaN_90> which edit?
[15:18] <psupreme> CDialog
[15:18] <BriaN_90> CButton?
[15:18] <BriaN_90> why
[15:19] <psupreme> jas told me
[15:19] <BriaN_90> uhm
[15:19] <psupreme> unless he fixed it
[15:19] <BriaN_90> there was a bug
[15:19] <BriaN_90> in the CDialog
[15:19] <BriaN_90> i wrote yestrday
[15:19] <BriaN_90> here on irc
[15:20] <BriaN_90> line 49 was
[15:20] <BriaN_90> draw(0)
[15:20] <BriaN_90> where fardad told me that it has to be draw(C_FULL_FRAME)
[15:20] <BriaN_90> and that was the bug
[15:20] <BriaN_90> that didn't make work my CValEdit
[15:20] <psupreme> rgr
[15:21] <BriaN_90> what's rgr lol
[15:21] <psupreme> roger
[15:21] <BriaN_90> lol
[15:21] <psupreme> im going to save my changes to a diff file
[15:21] <psupreme> and commit
[15:21] <psupreme> it works, just need to remove [ ] when chaning
[15:21] <psupreme> changin
[15:22] <psupreme> okay
[15:22] <psupreme> update
[15:22] <psupreme> cbutton_new.cpp
[15:23] <psupreme> works with border
[15:23] <psupreme> not without
[15:23] <psupreme> yet
[15:23] <BriaN_90> ok let me see
[15:27] <BriaN_90> ok i fixed it
[15:27] <BriaN_90> now it works
[15:27] <BriaN_90> even without the borders
[15:27] <psupreme> yeh me too
[15:27] <psupreme> was about to commit
[15:27] <psupreme> ok
[15:27] <BriaN_90> look mine
[15:27] <BriaN_90> see if it's the same
Session Close: Sat Nov 19 15:30:08 2011

Session Start: Sat Nov 19 15:30:08 2011
Session Ident: psupreme
[15:33] <BriaN_90> sorry i got disconnected
[15:33] <BriaN_90> did u say anything?
[15:33] <BriaN_90> <psupreme> yeh me too
[15:33] <BriaN_90> <psupreme> was about to commit
[15:33] <BriaN_90> <psupreme> ok
[15:33] <BriaN_90> <BriaN_90> look mine
[15:33] <BriaN_90> <BriaN_90> see if it's the same
[15:33] <BriaN_90> that's what i have last
[15:34] <psupreme> (15:28:52) (psupreme) console.display("[",CFrame::absRow(),CFrame::absCol()-1);
[15:34] <psupreme> (15:28:52) (psupreme) console.display("]",CFrame::absRow(),CFrame::absCol()+width()-4);
[15:34] <psupreme> (15:28:52) (psupreme) console.setPos(absRow(),absCol());
[15:34] <psupreme> (15:28:53) (psupreme) that
[15:34] <psupreme> (15:29:21) (psupreme) also
[15:34] <psupreme> (15:29:51) (psupreme) instead of "displaying" brackets, why not just allocate a string 2+stringlen, and put the brackets in the string directly
[15:34] <psupreme> (15:30:04) (psupreme) and overwrite if not selected
[15:34] <psupreme> (15:30:08) (psupreme) to ' '
[15:34] <psupreme> (15:30:56) (psupreme) also
[15:34] <psupreme> (15:30:58) (psupreme) change your set
[15:34] <psupreme> (15:31:04) (psupreme) to if(_data) delete ...
[15:34] <BriaN_90> i asked him about this
[15:34] <BriaN_90> and it said
[15:34] <BriaN_90> that if we display the brackets
[15:34] <BriaN_90> it's way faster
[15:34] <BriaN_90> that allocate and deallocate
[15:34] <BriaN_90> than
[15:35] <psupreme> not allocating and deallocating
[15:35] <psupreme> allocate once
[15:35] <psupreme> a string
[15:35] <psupreme> to hold stringlen+null+2 brackets
[15:36] <BriaN_90> and display the entire string?
[15:36] <psupreme> im tryign to whip up an example
[15:36] <BriaN_90> ok
[15:42] <psupreme> damn your tab spacing
[15:42] <BriaN_90> it's 2 characters
[15:42] <BriaN_90> that's what he said
[15:42] <BriaN_90> lol
[15:42] <psupreme> so ugly
[15:43] <BriaN_90> i know :(
[15:47] <psupreme> git it workin
[15:47] <psupreme> instead of calling display a bunch of times
[15:47] <psupreme> just gotta do something like
[15:47] <psupreme>         ((char*)_data)[0] = '[';
[15:47] <psupreme>         ((char*)_data)[width()-3] = ']';
[15:48] <BriaN_90> but then data
[15:48] <BriaN_90> will be always that
[15:48] <psupreme> nope
[15:48] <psupreme> just overwrite it if not selected
[15:48] <psupreme> right?
[15:49] <BriaN_90> yes but after?
[15:49] <BriaN_90> we need to fix data again
[15:49] <psupreme> for button, data is never used for anything else but the string of the button
[15:50] <BriaN_90> uhm
[15:51] <psupreme> and the set function handles everything for us
[15:51] <psupreme> it allocates strlen+3
[15:51] <psupreme> fills it
[15:51] <psupreme> and its good
[15:53] <BriaN_90> so u r saying
[15:53] <BriaN_90> to reallocate the data
[15:53] <BriaN_90> in the edit?
[15:53] <BriaN_90> adding the brackets?
[15:56] <psupreme> he doesnt even have a test for button edit
[15:57] <BriaN_90> yes he does
[15:57] <BriaN_90> test 4
[15:57] <psupreme> im running it
[15:57] <psupreme> you cant edit the data of the button
[15:57] <psupreme> like the text
[15:58] <BriaN_90> nope u cant
[15:58] <BriaN_90> the edit is to get the key
[15:58] <BriaN_90> that is pressed
[15:58] <BriaN_90> and switch between buttons
[15:58] <psupreme> weird naming
[15:58] <BriaN_90> yes lol
[15:59] <BriaN_90> btw i have to go now
[15:59] <BriaN_90> i will finish it tonight hopefully
[15:59] <BriaN_90> let's see if i can make it work :(
[15:59] <psupreme> i can get it finished for you?
[15:59] <BriaN_90> if u want
[15:59] <BriaN_90> we can split the work on the wiki
[15:59] <BriaN_90> i don't wanna get the credit
[15:59] <BriaN_90> for something that i did half
[16:00] <psupreme> ??
[16:01] <BriaN_90> i don't wanna get the full mark for it
[16:01] <BriaN_90> if u did half
[16:01] <BriaN_90> so we will fix the wiki page
[16:01] <BriaN_90> gotta go
[16:02] <BriaN_90> let me know if u do something
[16:02] <BriaN_90> bye
[16:02] <psupreme> k
[16:41] <psupreme> okay
[16:41] <psupreme> button 100% complete and working
[20:39] <psupreme> hey
[20:39] <psupreme> you see the updated button i did?
Session Close: Sat Nov 19 21:41:37 2011

Session Start: Sat Nov 19 15:30:08 2011
Session Ident: psupreme

Session Start: Sat Nov 19 21:41:37 2011
Session Ident: psupreme
[22:51] <BriaN_90> hey
[22:51] <BriaN_90> yes i saw it
[22:52] <psupreme> see what i did
[22:52] <psupreme> and why?
[22:52] <BriaN_90> there is a bug
[22:52] <BriaN_90> the cursor should be
[22:52] <BriaN_90> under the first letter
[22:53] <BriaN_90> i can fix it
[22:53] <BriaN_90> ok
[22:53] <BriaN_90> i fixed it
[22:54] <psupreme> its in my branch
[22:54] <BriaN_90> yes i'm using your code
[22:55] <psupreme> oh ok, thought you didnt update
[22:55] <BriaN_90> i did
[22:55] <BriaN_90> last test
[22:55] <BriaN_90> and then we can merge i think
[22:55] <psupreme> ok
[22:56] <BriaN_90> we can merge directly your CButton
[22:56] <BriaN_90> it's better than mine
[22:57] <psupreme> you know how it works right?
[22:57] <BriaN_90> ok now i fixed your CButton
[22:57] <BriaN_90> i think yes
[22:57] <BriaN_90> i have this solution in my idea
[22:57] <BriaN_90> but
[22:57] <psupreme> i explained it in the set(...)
[22:57] <BriaN_90> he told me that displaying directly
[22:58] <BriaN_90> the []
[22:58] <BriaN_90> was more efficient
[22:58] <BriaN_90> but who cares lol
[22:58] <psupreme> Id say this way is faster
[22:58] <psupreme> much more efficient than a function call to the display function
[22:59] <BriaN_90> the only thing
[22:59] <BriaN_90> calling a method form the constructor is slow
[22:59] <psupreme> the set method?
[22:59] <BriaN_90> yes
[22:59] <BriaN_90> it would be better
[22:59] <psupreme> nicer than rewriting the code
[22:59] <BriaN_90> if the constructor
[22:59] <BriaN_90> do exactly the same thing of set
[22:59] <BriaN_90> without calling it
[22:59] <BriaN_90> yes is nices
[22:59] <psupreme> I bet the compiler just optimizes that in anyway
[23:00] <BriaN_90> but slowe in perfomances
[23:00] <BriaN_90> yes probably
[23:00] <BriaN_90> right now we don't need that much speed
[23:00] <psupreme> If the function was called thousands of times, i would agree
[23:00] <BriaN_90> so we can leave it like this i suppose
[23:00] <psupreme> but its not really neccisary
[23:00] <BriaN_90> yes
[23:01] <BriaN_90> hm my english sux but
[23:01] <BriaN_90> Clear any junk if he have any
[23:01] <BriaN_90> is that right?
[23:01] <BriaN_90> should that be has?
[23:01] <psupreme> oops
[23:01] <psupreme> should be
[23:01] <psupreme> "if we have any"
[23:01] <BriaN_90> ok lol
[23:01] <BriaN_90> i can fix it
[23:01] <BriaN_90> ok
[23:01] <BriaN_90> i think we are ready to merge
[23:02] <BriaN_90> and see what happen
[23:02] <psupreme> yes
[23:02] <psupreme> ill merge my checkmark
[23:02] <BriaN_90> and merge CButton too
[23:02] <BriaN_90> when u r done
[23:02] <BriaN_90> i will merge my CValEdit
[23:04] <psupreme> I have to manually merge
[23:04] <psupreme> since i edited some files that should NOT be merged
[23:04] <BriaN_90> yes i have to do the same thing
[23:04] <BriaN_90> i messed up some files i think lol
[23:04] <BriaN_90> i will merge only
[23:04] <BriaN_90> the two files
[23:04] <BriaN_90> CValEdit
[23:04] <psupreme> also, make sure files all lowercase
[23:05] <BriaN_90> ye i fixed them
[23:05] <psupreme> too keep it consistent
[23:06] <psupreme> okay, added
[23:07] <psupreme> you merge now
[23:07] <BriaN_90> ok
[23:07] <BriaN_90> oh shit
[23:07] <BriaN_90> i have to comment my cvaledit first
[23:07] <BriaN_90> lol
[23:07] <psupreme> we'll have to manually change the vcxproj too
[23:07] <BriaN_90> i hate commenting
[23:07] <BriaN_90> yes
[23:07] <psupreme> you dont have to now
[23:07] <BriaN_90> ok i will do it after
[23:09] <BriaN_90> how can u merge only two files?
[23:09] <psupreme> I copy pasted
[23:09] <BriaN_90> oh
[23:10] <BriaN_90> i thought it would ruin the integrity if i do copy paste
[23:10] <psupreme> try to branch only 2 files
[23:10] <psupreme> nvm
[23:10] <BriaN_90> yes let me try first
[23:12] <BriaN_90> it doesn't work
[23:13] <BriaN_90> i think we have to copy paste lol
[23:13] <psupreme> yes
[23:13] <psupreme> did hex finish his aswell?
[23:14] <BriaN_90> i think
[23:14] <BriaN_90> yes
[23:17] <psupreme> ok
[23:17] <psupreme> update
[23:17] <psupreme> i added all tests aswell
[23:17] <BriaN_90> ok
[23:17] <BriaN_90> we should update
[23:17] <BriaN_90> the vcxproj
[23:17] <BriaN_90> i can do it
Session Close: Sat Nov 19 23:17:55 2011

Session Start: Sat Nov 19 23:17:55 2011
Session Ident: psupreme
�02[23:18] * Not connected to server
[23:19] <BriaN_90> sorry my connection
[23:19] <BriaN_90> i am updating the vcxproj
[23:19] <psupreme> wait
[23:19] <BriaN_90> when jas are going to merge?
[23:20] <psupreme> ok
[23:20] <psupreme> update
[23:20] <psupreme> files should be all lowercase
[23:20] <psupreme> hex's files
[23:20] <BriaN_90> ye i was going to do it
[23:20] <psupreme> hex's comments so...... useful?
[23:21] <BriaN_90> dunno lol
[23:21] <psupreme> jas had a major bug with his code
[23:21] <psupreme> and he has to redo it
[23:21] <BriaN_90> ok
[23:21] <psupreme> some heap corruption bug
[23:21] <BriaN_90> omg
[23:21] <BriaN_90> i hope he can do it
[23:21] <BriaN_90> btw
[23:21] <psupreme> since he was using all old versions of our code
[23:21] <BriaN_90> which test should i include in the vcxproj?
[23:22] <psupreme> and all bugs were still present
[23:22] <psupreme> uh
[23:22] <psupreme> none?
[23:22] <BriaN_90> yes sounds fine for me
[23:22] <BriaN_90> and i have to fix
[23:22] <psupreme> hmm
[23:22] <psupreme> maybe one of them tho
[23:22] <BriaN_90> the bug in the CDialog::edit line 49
[23:22] <BriaN_90> i'm fixing it right now
[23:23] <BriaN_90> and i will commit the trunk
[23:23] <BriaN_90> without testers for the moment
[23:23] <BriaN_90> ok i will use all the test
[23:24] <BriaN_90> for the moment
[23:24] <BriaN_90> to check if everything works
[23:24] <psupreme> wont compile, each one has a main( )
[23:24] <BriaN_90> lol
[23:24] <BriaN_90> i just add
[23:24] <BriaN_90> one at time
[23:24] <BriaN_90> :P
[23:24] <psupreme> if you want..
[23:24] <BriaN_90> try to run
[23:24] <BriaN_90> test 3
[23:25] <BriaN_90> it doesn't compile
[23:25] <BriaN_90> there are problem with the cbutton
[23:25] <BriaN_90> and ccheckmark
[23:25] <psupreme> can you commit vcxproj
[23:25] <psupreme> yes
[23:25] <BriaN_90> ok let me commit
[23:26] <psupreme> ok ill fix the bugs for both classes
[23:26] <BriaN_90> ok
[23:26] <BriaN_90> in the meanwhile i will try to run the other test
[23:26] <BriaN_90> BUT
[23:26] <BriaN_90> they won't compile of course lol
[23:26] <BriaN_90> i'll wait for your bug fixes
[23:27] <psupreme> oh right, i think the cc has to be public
[23:27] <BriaN_90> we should post this conversation for the other guys
[23:27] <psupreme> yes
[23:27] <psupreme> CLabel copy ctor has to be publuic
[23:27] <psupreme> i believe
[23:27] <BriaN_90> uhm
[23:27] <psupreme> and
[23:27] <psupreme> wow
[23:27] <BriaN_90> oh wait
[23:27] <psupreme> what happend to CButton
[23:27] <BriaN_90> i didn't commit
[23:28] <BriaN_90> the clabel
[23:28] <BriaN_90> the new one
[23:28] <BriaN_90> uhm
[23:28] <psupreme> ctrl+c, v
[23:28] <BriaN_90> yes
[23:28] <psupreme> wow
[23:28] <psupreme> how did Cbutton break..
[23:29] <BriaN_90> omg
[23:29] <psupreme> yeh
[23:29] <psupreme> i fixed
[23:29] <BriaN_90> ok let me update
[23:29] <BriaN_90> so wait
[23:30] <BriaN_90> the copy ctor should be public?
[23:30] <BriaN_90> yes
[23:30] <BriaN_90> :D
[23:30] <psupreme> yes
[23:31] <psupreme> okay, button fixed
[23:31] <BriaN_90> ok let me run everything
[23:31] <BriaN_90> and commit
[23:32] <psupreme> and, im going to fix all indenting and var names
[23:32] <psupreme> param names should be camelCase
[23:32] <psupreme> not
[23:32] <psupreme> Camelcaseshit
[23:33] <BriaN_90> ok now everything works
[23:33] <BriaN_90> try
[23:33] <BriaN_90> i committed
[23:34] <BriaN_90> now i am going to test all testers
[23:34] <psupreme> for label
[23:34] <psupreme> i think you can do
[23:34] <psupreme> L.row()
[23:34] <psupreme> instead of L.CFrame::row()
[23:34] <BriaN_90> i like to use the full name
[23:34] <BriaN_90> because when we are
[23:34] <psupreme> since its not virtual
[23:35] <BriaN_90> when we have many classes inherited
[23:35] <BriaN_90> it's better to see from which class i'm getting it
[23:35] <BriaN_90> it's my convention lol
[23:35] <psupreme> too much writing for me
[23:36] <BriaN_90> ye i know :P
[23:37] <BriaN_90> there is a bug
[23:37] <BriaN_90> in the test 5
[23:37] <BriaN_90> it is trying to include
[23:37] <BriaN_90> cveditline.h
[23:37] <BriaN_90> this file doesn't exist
[23:37] <psupreme> lolwtf
[23:38] <BriaN_90> maybe it's a typo
[23:38] <BriaN_90> i'll fix it
[23:38] <psupreme> okay update
[23:38] <psupreme> fixed label formatting
[23:38] <BriaN_90> it should be cvaledit
[23:40] <psupreme> good work on valedit
[23:40] <BriaN_90> thanks
[23:41] <psupreme> works nice with fixed test
[23:41] <BriaN_90> ok i think everything works fine
[23:41] <BriaN_90> i committed
[23:42] <BriaN_90> i removed the testers
[23:42] <BriaN_90> the project has only the classes, no mains
[23:42] <psupreme> yes thats good
[23:42] <BriaN_90> when do u think jas will merge?
[23:42] <BriaN_90> we need to tag before tomorrow at 15.00
[23:43] <BriaN_90> if he needs help he can ask
[23:43] <BriaN_90> i see that he doesn't commit a lot
[23:43] <BriaN_90> lol
[23:43] <psupreme> he commits whole proj
[23:43] <psupreme> instead of incrimently
[23:44] <BriaN_90> we need to increase our revision
[23:44] <BriaN_90> is too low
[23:44] <psupreme> naw
[23:44] <BriaN_90> people have up to 300
[23:44] <psupreme> we're just too good and make less mistakes
[23:45] <BriaN_90> no we are not
[23:45] <BriaN_90> :P
[23:45] <psupreme> :(
[23:45] <BriaN_90> ok well i think our work is done for tonight
[23:45] <BriaN_90> do u want me to post the conversation?
[23:45] <psupreme> if you want
[23:45] <BriaN_90> ok